-
Notifications
You must be signed in to change notification settings - Fork 432
RI-7976: output SHA-512 checksums in GitHub workflow summary #5419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Code Coverage - Backend unit tests
Test suite run success3056 tests passing in 290 suites. Report generated by 🧪jest coverage report action from 99f7fa5 |
There was a problem hiding this 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.
Contributor
Code Coverage - Integration Tests
|
Contributor
Code Coverage - Frontend unit tests
Test suite run success5553 tests passing in 708 suites. Report generated by 🧪jest coverage report action from 99f7fa5 |
pawelangelow
previously approved these changes
Jan 30, 2026
valkirilov
reviewed
Jan 30, 2026
valkirilov
previously approved these changes
Jan 30, 2026
Add script to parse electron-builder YAML files and display checksums in the GitHub Actions job summary for easy copy-paste to release notes. Eliminates need to manually navigate AWS S3 to retrieve checksum values. References: #RI-7976
99f7fa5
452b370 to
99f7fa5
Compare
valkirilov
approved these changes
Feb 2, 2026
pawelangelow
approved these changes
Feb 2, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
generate-checksums-summary.jsscript to parse electron-builder YAML files (latest.yml,latest-linux.yml,latest-mac.yml)aws-upload-dev.yml) and production (aws-upload-prod.yml) workflowsThis eliminates the need to manually navigate AWS S3 to retrieve checksum values for release notes.
Test plan
release/*branch)Note
Low Risk
Low risk: adds a non-blocking workflow summary step that parses build metadata and does not affect build artifacts or S3 upload logic. Main risk is missing/incorrect parsing if electron-builder YAML format changes, but
continue-on-errorprevents workflow failures.Overview
Adds a new
.github/generate-checksums-summary.jsscript that parses electron-builderlatest*.ymlfiles inrelease/and appends a SHA-512 checksums Markdown table to the GitHub Actions job summary (mapped to the release-notes package names and sorted consistently).Integrates this into both
aws-upload-dev.ymlandaws-upload-prod.ymlas acontinue-on-errorstep so checksum reporting is best-effort and won’t block uploads.Written by Cursor Bugbot for commit 99f7fa5. This will update automatically on new commits. Configure here.