Skip to content

fix(ci): Bash syntax error in GitHub Actions - #371

Merged
hangy merged 3 commits into
mainfrom
fix-ci-error
May 30, 2026
Merged

fix(ci): Bash syntax error in GitHub Actions#371
hangy merged 3 commits into
mainfrom
fix-ci-error

Conversation

@hangy

@hangy hangy commented May 23, 2026

Copy link
Copy Markdown
Member

What

Fix bash syntax error in cosign stage.

Screenshot

N/A

Fixes bug(s)

Part of

N/A

@hangy hangy self-assigned this May 23, 2026
@hangy
hangy enabled auto-merge (squash) May 23, 2026 08:27
@codecov-commenter

codecov-commenter commented May 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.33%. Comparing base (15175a6) to head (d361dd8).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #371   +/-   ##
=========================================
  Coverage     43.33%   43.33%           
  Complexity       58       58           
=========================================
  Files            18       18           
  Lines           450      450           
  Branches         46       46           
=========================================
  Hits            195      195           
  Misses          227      227           
  Partials         28       28           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hangy hangy changed the title fix(ci): YAML error in GitHub Actions fix(ci): Bash syntax error in GitHub Actions May 23, 2026
@hangy
hangy merged commit e17114e into main May 30, 2026
13 checks passed
@hangy
hangy deleted the fix-ci-error branch May 30, 2026 19:18
@john-gom

Copy link
Copy Markdown
Collaborator

@hangy I think the deploy is now failing because the job that it waits for is renamed due to the new matrix strategy. I'm away from my desk until this evening so will try and figure it out then unless you have time to look at it today. Cc @harsh-791

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants