Skip to content

ref(ci): change update-package-sha.sh to use loop approach#7916

Merged
philprime merged 2 commits into
mainfrom
philprime/refactor-update-package-sha
May 19, 2026
Merged

ref(ci): change update-package-sha.sh to use loop approach#7916
philprime merged 2 commits into
mainfrom
philprime/refactor-update-package-sha

Conversation

@philprime
Copy link
Copy Markdown
Member

@philprime philprime commented May 19, 2026

This pull request refactors the checksum update logic in the scripts/update-package-sha.sh script to make it more maintainable and robust. Instead of using hardcoded and repetitive commands for each framework variant, the script now uses a loop over a structured list of zip files and their corresponding markers, improving scalability and reducing the likelihood of errors.

Extracted from #7598

#skip-changelog

@philprime philprime self-assigned this May 19, 2026
@philprime philprime added the ready-to-merge Use this label to trigger all PR workflows label May 19, 2026
@philprime philprime enabled auto-merge (squash) May 19, 2026 09:38
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 89527fd. Configure here.

Comment thread scripts/update-package-sha.sh
Copy link
Copy Markdown
Contributor

@NinjaLikesCheez NinjaLikesCheez left a comment

Choose a reason for hiding this comment

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

LGTM

@philprime philprime merged commit 5eef2b9 into main May 19, 2026
143 checks passed
@philprime philprime deleted the philprime/refactor-update-package-sha branch May 19, 2026 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants