Skip to content

Add checksums to download table#80

Merged
timtmok merged 5 commits intomainfrom
show-download-checksums
Jun 6, 2025
Merged

Add checksums to download table#80
timtmok merged 5 commits intomainfrom
show-download-checksums

Conversation

@timtmok
Copy link
Collaborator

@timtmok timtmok commented Jun 2, 2025

Address posit-dev/positron#7317

One part of the changes to show the sha-256 hashes. This only displays if the checksum file is available for the release. Clicking on the checksum copies the full checksum to the clipboard.

image

The checksum file is generated during the upload to s3 build stage when the release metadata is generated.

After this change is on the website, I can add the checksum file manually for 2025.06.0-167 and the checksums will appear.

@netlify
Copy link

netlify bot commented Jun 2, 2025

Deploy Preview for positron-posit-co ready!

Name Link
🔨 Latest commit ca8ad38
🔍 Latest deploy log https://app.netlify.com/projects/positron-posit-co/deploys/68431a0de6841d0008e0b0e0
😎 Deploy Preview https://deploy-preview-80--positron-posit-co.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@timtmok
Copy link
Collaborator Author

timtmok commented Jun 3, 2025

Waiting on https://github.com/posit-dev/positron-builds/pull/391 to merge before this ready for review in case there are changes to where the checksum file is hosted or its format.

timtmok added 2 commits June 5, 2025 15:13
Shows only if available
Clicking on checksum copies it to clipboard
@timtmok timtmok force-pushed the show-download-checksums branch 2 times, most recently from 1af70ac to ec95992 Compare June 6, 2025 13:26
Access checksums by filename
@timtmok timtmok force-pushed the show-download-checksums branch from ec95992 to 36b31e4 Compare June 6, 2025 14:00
@timtmok
Copy link
Collaborator Author

timtmok commented Jun 6, 2025

I've added the checksums file to the prereleases and updated the freeze file. The preview now shows the checksums in the downloads table.

@timtmok timtmok marked this pull request as ready for review June 6, 2025 14:12
@timtmok timtmok requested a review from juliasilge June 6, 2025 14:12
Copy link
Collaborator

@juliasilge juliasilge left a comment

Choose a reason for hiding this comment

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

Thank you so much! 🙌

@timtmok timtmok merged commit 3baaad9 into main Jun 6, 2025
5 checks passed
@timtmok timtmok deleted the show-download-checksums branch June 6, 2025 17:09
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