Skip to content
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

Update download links and blog post for Pulsar v1.119.0 #277

Merged
merged 2 commits into from
Jul 19, 2024

Conversation

DeeDeeG
Copy link
Member

@DeeDeeG DeeDeeG commented Jul 17, 2024

Here's the blog post and download link updates for Pulsar 1.119.0!

Blog content by confused-Techie, with copy-editing suggestions from Daeraxa and technical clarification from savetheclocktower.

"Notice" section by confused-Techie, with an edit for clarification by me (DeeDeeG), plus feedback from savetheclocktower.

Blog content by confused-Techie, with copy-editing suggestions
from Daeraxa and technical clarification from savetheclocktower.

"Notice" section by confused-Techie, with an edit for clarification
by me (DeeDeeG), plus feedback from savetheclocktower.

Co-authored-by: confused_techie <[email protected]>
Co-authored-by: Daeraxa <[email protected]>
Co-authored-by: Andrew Dupont <[email protected]>
Comment on lines 25 to 34
---

## **NOTICE**

Originally, the binaries from our normal "Pull Request" CI build (which produces unsigned binaries) were accidentally uploaded to this release, instead of the binaries from the special "tag push" CI build (which signs the macOS binaries). In order to provide you with signed macOS binaries, we are re-uploading the Intel macOS binaries and updating `SHA256SUMS.txt` to reflect this. As such the following binaries have been swapped out for the correct versions, with the checksum being updated as well for the following files:

- `Intel.Mac.Pulsar-1.119.0-mac.zip`
- `Intel.Mac.Pulsar-1.119.0.dmg`

---
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
---
## **NOTICE**
Originally, the binaries from our normal "Pull Request" CI build (which produces unsigned binaries) were accidentally uploaded to this release, instead of the binaries from the special "tag push" CI build (which signs the macOS binaries). In order to provide you with signed macOS binaries, we are re-uploading the Intel macOS binaries and updating `SHA256SUMS.txt` to reflect this. As such the following binaries have been swapped out for the correct versions, with the checksum being updated as well for the following files:
- `Intel.Mac.Pulsar-1.119.0-mac.zip`
- `Intel.Mac.Pulsar-1.119.0.dmg`
---
---
::: info Notice
Originally, the binaries from our normal "Pull Request" CI build (which produces unsigned binaries) were accidentally uploaded to this release, instead of the binaries from the special "tag push" CI build (which signs the macOS binaries).
In order to provide you with signed Intel macOS binaries, we are re-uploading them and updating `SHA256SUMS.txt` to reflect this.
The following binaries have been swapped out for the correct versions and the checksums updated:
- `Intel.Mac.Pulsar-1.119.0-mac.zip`
- `Intel.Mac.Pulsar-1.119.0.dmg`
:::

We should probably use an infobox here to draw attention to the notice. I've also attempted a little rewording and formatting to reduce some repetition and make it a bit easier to read.

Copy link
Member Author

Choose a reason for hiding this comment

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

I committed it with a rephrase of the present-tense "we are re-uploading" stuff to past tense "we have re-uploaded", and so on. And removed one of the separator --- lines, since in my view the Notice infobox belongs with the main prose passage, IMO, not the bullet points of the "individual changes" section.

@Daeraxa
Copy link
Member

Daeraxa commented Jul 19, 2024

LGTM! Thanks for putting it all together

@DeeDeeG
Copy link
Member Author

DeeDeeG commented Jul 19, 2024

Merging, blog post should be live in a couple of minutes!

@DeeDeeG DeeDeeG merged commit 542f857 into main Jul 19, 2024
1 of 2 checks passed
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