-
-
Notifications
You must be signed in to change notification settings - Fork 170
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
Switch Python 3.11 to stable release in CI #790
Conversation
CHANGES/790.misc.rst
Outdated
@@ -0,0 +1 @@ | |||
|
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.
An empty change note is probably not the best idea. Either fill it out or I'll apply a label for the bot to just ignore it. I'm actually not sure what Towncrier would do with an empty fragment...
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.
I can certainly remove it, but it is a normal option with towncrier. That way it's all 'in the system' rather than you having to add external tooling to sometimes ignore the check via a GitHub PR label as it sounds like you have. I'd link to the twisted contributing docs but... the link I found is 404.
You don't appear to be configuring custom sections in the changelog.
[tool.towncrier]
package = "yarl"
filename = "CHANGES.rst"
directory = "CHANGES/"
title_format = "{version} ({project_date})"
issue_format = "`#{issue} <https://github.com/aio-libs/yarl/issues/{issue}>`_"
So you would be using the defaults where, as I recall, misc
has showcontent
set to false.
showcontent
: Whether if the fragment contents should be included in the news file. If omitted, it defaults to true
So, you get just a list of the issue/pr/file numbers as at https://github.com/aio-libs/yarl/blob/68e7d9312ad9eccb695dbc174a4705e9a238c1fa/CHANGES.rst#misc-1. Note that if you follow those, their content isn't empty, just ignored. https://github.com/aio-libs/yarl/pull/647/files#diff-3c671c897bfbc0b283bbb224c8784fda9364a40d0233748d2606c6b78faa0394
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.
Fun, so GitHub let me edit the 'empty' file that actually had 'two lines' and delete one and commit it and it still seems to be the same file unchanged... But, you've got a linter complaining about it so I'll go ahead and delete it since that seems to be process here.
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.
Not to be pounding on this... but I got the doc link so may as well finalize the references here.
https://docs.twisted.org/en/latest/development/dev-process.html#release-notes-management
misc - Issues which are very minor and not worth summarizing outside of the git changelog. These should be empty (their contents will be ignored)
And the PR fixing the doc link is an example of an empty misc fragment.
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.
FWIW I've been considering showing that content at some point. Too many people misuse these fragments and put really useful stuff there. And it's hard to catch for humans during reviews.
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.
It's all in your intent in the setup. Nice and configurable.
To make fragments a bit more reviewable, at least in terms of viewing them rendered in case there's any markup errors, I added a step to dump the draft output to the GitHub workflow summary.
- name: Create single version changlog summary
run: |
towncrier build --yes --draft >> $GITHUB_STEP_SUMMARY
(not sure what the --yes
is doing there...)
Sample output from this at https://github.com/Chia-Network/chia-blockchain/actions/runs/3445535929#summary-9436412167.
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.
@altendky yea, I actually often make the integration with both Sphinx (via by extension @ https://github.com/sphinx-contrib/sphinxcontrib-towncrier — can error out in nitpicky mode) and GH job summary (requires pandoc for RST->md). See:
Codecov Report
@@ Coverage Diff @@
## master #790 +/- ##
=======================================
Coverage 99.73% 99.73%
=======================================
Files 4 4
Lines 750 750
Branches 212 208 -4
=======================================
Hits 748 748
Misses 2 2
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Side note, what do the |
The tilda variant has been suggested by the GitHub stuff on the issue tracker of |
What do these changes do?
Just tidying the now-released CPython 3.11
pyver
entry.Are there changes in behavior for the user?
No.
Related issue number
Adjacent to #781.