Skip to content

Small fix for syntax error in update-pricing workflow#153

Open
saengel wants to merge 1 commit into
mainfrom
bug/sc-44875/fix-broken-pricing-pipeline
Open

Small fix for syntax error in update-pricing workflow#153
saengel wants to merge 1 commit into
mainfrom
bug/sc-44875/fix-broken-pricing-pipeline

Conversation

@saengel

@saengel saengel commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

There was an unexpected EOF in the update-pricing workflow which caused it to fail, this PR fixes it.

@saengel saengel requested review from Copilot and yodem June 15, 2026 17:32
@gitvelocity-reviewer

Copy link
Copy Markdown

📊 Code Quality Score: 1/100

10 × 0.1 (Nano ESF) = 1.0

Category Score Factors
🔭 Scope 2/20 Single file, single CI workflow, no cross-cutting impact
🏗️ Architecture 1/20 No architectural change; minor workflow argument fix
⚙️ Implementation 2/20 Simple shell argument addition and trailing backslash removal; fixes real bug
⚠️ Risk 3/20 Low risk; workflow was likely broken/unreliable before; easily reversible
✅ Quality 2/15 No tests applicable for CI config; change is self-documenting
🔒 Perf / Security 0/5 No performance or security considerations

Was this score accurate? 👍 Yes · 👎 No

Scored by GitVelocity · How are scores calculated?

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes a GitHub Actions workflow failure in update-pricing by correcting the gh pr create invocation so the multi-line command is properly terminated (addressing the previously broken line continuation).

Changes:

  • Fix gh pr create line continuation to avoid malformed shell syntax.
  • Add an explicit base branch argument for PR creation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +46 to 50
--base main \
--head "$BRANCH"
fi
env:
GH_TOKEN: ${{ github.token }}
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.

2 participants