Skip to content

Fix block new pull requests workflow#30017

Merged
JesperSchulz merged 1 commit into
mainfrom
fix/block-new-prs-workflow-yaml
May 6, 2026
Merged

Fix block new pull requests workflow#30017
JesperSchulz merged 1 commit into
mainfrom
fix/block-new-prs-workflow-yaml

Conversation

@JesperSchulz
Copy link
Copy Markdown
Contributor

@JesperSchulz JesperSchulz commented May 6, 2026

Summary

  • fix invalid YAML in BlockNewPullRequests.yml caused by an unindented multiline JavaScript template literal
  • run the PR blocker on pull_request_target with explicit issues: write and pull-requests: write permissions so it can comment on and close new PRs
  • quote $GITHUB_OUTPUT writes and set shell: bash for the date comparison step

Validation

  • parsed .github/workflows/BlockNewPullRequests.yml with PyYAML successfully
  • confirmed the original run failure points to YAML syntax line 30

Correct YAML parsing for the github-script message and run the enforcement on pull_request_target with explicit write permissions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@JesperSchulz JesperSchulz requested a review from a team as a code owner May 6, 2026 08:32
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Could not find a linked ADO work item. Please link one by using the pattern 'AB#' followed by the relevant work item number. You may use the 'Fixes' keyword to automatically resolve the work item when the pull request is merged. E.g. 'Fixes AB#1234'

@JesperSchulz JesperSchulz merged commit 6def411 into main May 6, 2026
6 of 9 checks passed
@JesperSchulz JesperSchulz deleted the fix/block-new-prs-workflow-yaml branch May 6, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant