-
-
Notifications
You must be signed in to change notification settings - Fork 38
Update download links and add blog post for Pulsar v1.117.0 #273
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
Conversation
The blog post content for this release was written by confused-Techie. Co-authored-by: confused_techie <[email protected]>
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.
Apologies for not providing feedback and your reworked blurb.
I see no issue at all for swapping your work with what's here if you'd like, and the approval with both apply since as I see it both are effective to communicate the details, ones just a bit more refined lol.
Otherwise thanks for getting this PR put together!
To make the mother of all nitpicks, I'd break this up into paragraphs for ease of skimming…
…but that's just a suggestion. |
Yeah, it occurred to me a number of times to break it up, and I didn't feel it had to as the ideas flow into each-other so much, but some part of me was nudging to break it up nonetheless. I think it's a fair suggestion. |
@savetheclocktower what do you think about this?:
-->
Going back to some of the wording that the original had that I think was better. ("OS-level dark mode"). |
Either is fine with me. |
This edit is partly based on feedback from savetheclocktower. Co-authored-by: Andrew Dupont <[email protected]>
Tweaked the wording a bit, mostly as discussed above but not 100% exactly. Ready to merge, I suppose. Will do so if no objections in the next hour or three. |
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.
LGTM! Thanks again for getting it put together
Okay, with this merged, and CI having done its thing, this Regular release post is now LIVE --> https://pulsar-edit.dev/blog/20240520-confused-Techie-v1.117.0.html (A little late, but the Release itself was on time!!) |
Note: The blog post content for this Pulsar v1.117.0 release was written by confused-Techie. This is just posting that content here as a new blog post and updating the download links from 1.116.0 to 1.117.0.
Alternative blurb option
Also consider this alternate blurb re-worked by me, I posted this in the Discord and did not receive any outright feedback there, so I went with the existing blurb by @confused-Techie to be safer. But we have this as an option:
I'd be okay to go over this and receive suggestions/feedback, either here or on Discord, if there is a desire to go over the blog post wording with a finer-toothed comb. But it works as-is too (as @confused-Techie wrote it, which is how it is committed for this PR already). So this ^ text is just an alternate second draft to consider.