-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Revert "Revert to one release notes file (#10639)" #10656
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
Revert "Revert to one release notes file (#10639)" #10656
Conversation
The change to the release notes organization is only needed for the stable/0.25 branch longer term. Moving forward we'll have a different release notes strategy likely using the reno sphinx extension directly. In the short term this reverts to the previous state prior to Qiskit#10639 as while Qiskit#10639 was necessary for getting 0.25.1 released it did turn off our CI coverage testing that release notes would be built. Restoring this is important so we know from CI whether a change to the release notes works as part of the docs build. This reverts commit 0388d54.
One or more of the the following people are requested to review this:
|
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.
Sure, fine with me. We should manually copy the 0.25.1 release notes into the legacy file once they are generated.
Pull Request Test Coverage Report for Build 5896437030
💛 - Coveralls |
Pull Request Test Coverage Report for Build 5943633889
💛 - Coveralls |
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.
This needs the 0.25.1 / 0.44.1 release still.
Summary
The change to the release notes organization is only needed for the stable/0.25 branch longer term. Moving forward we'll have a different release notes strategy likely using the reno sphinx extension directly. In the short term this reverts to the previous state prior to #10639 as while #10639 was necessary for getting 0.25.1 released it did turn off our CI coverage testing that release notes would be built. Restoring this is important so we know from CI whether a change to the release notes works as part of the docs build.
Details and comments
This reverts commit 0388d54.