Skip to content

Fix gh-aw lockfile update workflow syntax#19300

Merged
trask merged 1 commit into
open-telemetry:mainfrom
trask:fix-gh-aw-lockfiles-case-pattern
Jul 22, 2026
Merged

Fix gh-aw lockfile update workflow syntax#19300
trask merged 1 commit into
open-telemetry:mainfrom
trask:fix-gh-aw-lockfiles-case-pattern

Conversation

@trask

@trask trask commented Jul 22, 2026

Copy link
Copy Markdown
Member

The Check patch step split a Bash case pattern across lines after |, causing the auto-update workflow to fail with:

syntax error near unexpected token `newline'

Keep the two allowed path patterns on the same logical line so Bash can parse the allowlist and proceed with the generated lockfile patch.

This was exposed by #19216 when its build-common.yml update triggered the gh-aw lockfile workflow.

Validation:

  • Extracted Check patch run block passes bash -n
  • git diff --check

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 22, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-22 21:20:57 UTC.

  • Status: Merged.

This automated status or its linked feedback items may be incorrect. If something looks wrong, please report it with the result you expected.

@trask
trask marked this pull request as ready for review July 22, 2026 20:53
@trask
trask requested a review from a team as a code owner July 22, 2026 20:53
Copilot AI review requested due to automatic review settings July 22, 2026 20:53

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 Bash parsing in the gh-aw lockfile auto-update workflow.

Changes:

  • Keeps both allowed lockfile path patterns in one valid case pattern.

@trask
trask enabled auto-merge (squash) July 22, 2026 21:11
@trask
trask merged commit cb9be4a into open-telemetry:main Jul 22, 2026
185 of 187 checks passed
@trask
trask deleted the fix-gh-aw-lockfiles-case-pattern branch July 22, 2026 21:19
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.

3 participants