Skip to content

🐛 fix(release): generate docstrfmt-compatible changelog entries#463

Merged
gaborbernat merged 1 commit intotox-dev:mainfrom
gaborbernat:release
Mar 5, 2026
Merged

🐛 fix(release): generate docstrfmt-compatible changelog entries#463
gaborbernat merged 1 commit intotox-dev:mainfrom
gaborbernat:release

Conversation

@gaborbernat
Copy link
Member

The release workflow generates RST section headers for new changelog entries that docstrfmt immediately reformats, causing pre-commit.ci to fail on every release commit. This happened on 4.9.4 (pre-commit.ci run) and was silently auto-fixed on earlier releases (4.9.2, etc.) by subsequent pre-commit.ci autoupdate PRs.

🔧 Two formatting issues: the * overline/underline separator matched the title length exactly, but docstrfmt expects it to be one character wider (to account for the leading space in RST overlined headers). Additionally, no blank line was emitted between the section underline and the bullet list. Both are now corrected in the printf format string.

A secondary bug is also fixed: _load_excluded_authors in tasks/changelog.py still referenced .github/release.yml, which was renamed to .github/release.yaml in b10b8c5. This caused bot commits (dependabot[bot], pre-commit-ci[bot]) to appear in the 4.9.4 changelog instead of being filtered out.

The release workflow generated RST section headers that docstrfmt
reformats: overline/underline was same width as title (docstrfmt
wants +1), and the blank line after the underline was missing.

Also fix _load_excluded_authors to use .yaml extension after the
.github files were renamed in b10b8c5.
@gaborbernat gaborbernat requested a review from ofek as a code owner March 5, 2026 18:59
@gaborbernat gaborbernat added the bug Something isn't working label Mar 5, 2026
@gaborbernat gaborbernat enabled auto-merge (squash) March 5, 2026 19:00
@gaborbernat gaborbernat merged commit 6c134d3 into tox-dev:main Mar 5, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant