Conversation
certbot/CHANGELOG.md
Outdated
| This release was only done for our snaps, however, all of our distribution | ||
| mechanisms were updated. |
There was a problem hiding this comment.
I have trouble understanding what the second half of this sentence means. Like, do non-snap users need to care?
There was a problem hiding this comment.
This release wouldn't be important to non-snap users, however, I was planning on pushing an update everywhere for simplicity. If you look at b5503e9 or my crossed out text in Mattermost, I was initially planning on limiting the release to snap & docker users, but I'm hesitant to change our release process and do release code in a way we've never done before right before I disappear for a week and a half. We could revisit that plan or we could try to clarify this text to flag that this release is really only useful to our snap users. What do you think?
There was a problem hiding this comment.
Ah, I misread it as "we're only releasing the snap", rather than "this release was only done [for the sake of] our snaps".
I think clarifying the text would be useful. 👍
There was a problem hiding this comment.
Ah OK. Sounds good.
Feel free to offer an alternative if you still don't like the new text.
See https://opensource.eff.org/eff-open-source/pl/8tepfnrdt7r5jxryz43d6y9z6r for more info.
The changelog says
masterbecause it is changed to the current date in the release script. This is what we've done before in PRs like #7511.