-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Upcoming whatsnews can't be tested on RTD CI #2074
Comments
FWIW, I don't consider it crucial that whatsnew entries render perfectly when each PR gets merged. That's because we can always clean it up in the "finalize release" PR, when we are already making changes to that file anyway (updating release date, listing missed contributors, etc). So as far as reviewing the whatsnew entry in each PR, I personally don't mind just checking the un-rendered RST, and it's no big deal if we overlook some syntax issue, since we have an easy opportunity to fix it later. However, even if it's not required, I agree that it is nice to have the upcoming whatsnew page included in Anyway, that's just a bit of background information. I don't feel strongly one way or the other :) |
Thanks for the context, thou I was aware of the last-minute fixes. It's also just to leverage future work and allow us contributors to be equally functional at checking the whatsnew. You know I'm kinda a fan of leveraging maintainer tasks. Feel free to close this issue even if no action is deemed to be taken. I'll leave you a comment here below regarding the release procedure wiki entry. I find it overly complex, and if I were to update it, I would Before any [pre-]releaseCheck thingies Pre-releasescheck before any release, and do a PR with fixes Releasescheck before any release, and do a PR with fixes From GitHub (recommended or not, who knows)From local branchwhat to do if CI fails at uploading the new version
Update
|
I support adding an extra step to the release procedure, where the responsible person commits a PR right after a new release that adds the whatsnew page. |
I've made some improvements to the release procedure along these lines, including the main suggestion of including the new whatsnew page in the docs immediately after the release :) https://github.com/pvlib/pvlib-python/wiki/Release-procedures Thanks for pursuing these improvements, @echedey-ls! |
That's amazing @kandersolar !! Feel free to close this issue at your will 😄 |
Problem
Adding a version whatsnew to the whatsnews index at the end of a development cycle makes it difficult to test the rendering of the whatsnews page.
Also, it makes checking the
latest
ReadTheDocs whatsnews page useless, since it won't have the latest changes entries.Describe the solution you'd like
Modify release instructions to add the next whatsnew file to the whatsnews page on a commit right after the tagged commit.
Describe alternatives you've considered
None. We can't add that file in the same version release PR since it would show up in the stable build.
The other alternative is to continue checking that locally (I really dislike that, I need to switch branches in an smaller timespan than a PR review 🤠)
Secondary effects
Next release schedule will show up in the latest docs branch, which some users may find useful.
The text was updated successfully, but these errors were encountered: