Skip to content

Conversation

@Crunchyman-ralph
Copy link
Collaborator

@Crunchyman-ralph Crunchyman-ralph commented Feb 6, 2026

Summary

  • Fix forward-port CI workflow failing with exit code 1
  • Replace grep -oE '[0-9]+$' with basename to extract PR number from URL
  • The $' sequence in the grep regex was being consumed by bash's ANSI-C quoting, breaking the command

Test plan

  • Trigger the forward-port workflow and verify the PR creation step succeeds
  • Verify PR_NUMBER is correctly extracted from the gh pr create output URL

Note

Low Risk
Small change limited to CI workflow scripting; main risk is mis-parsing unexpected gh output formats, but basename is simpler and less error-prone than the previous regex.

Overview
Fixes the forward-port workflow’s PR-number extraction by replacing the grep -oE '[0-9]+$' parsing with basename "$PR_URL", avoiding bash/regex quoting issues that could cause the job to fail and preventing follow-up steps (like labeling) from breaking.

Written by Cursor Bugbot for commit 2c4725f. This will update automatically on new commits. Configure here.

Replace grep regex with basename to avoid $' ANSI-C quoting
issue that caused the CI step to fail with exit code 1.
@changeset-bot
Copy link

changeset-bot bot commented Feb 6, 2026

⚠️ No Changeset found

Latest commit: 2c4725f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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