Skip to content
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

🐛 Fix RST in changelog template before links #7348

Conversation

webknjaz
Copy link
Member

@webknjaz webknjaz commented Jul 8, 2023

In corner cases, changelog fragments with things like detached link definitions (example: #7346) cause RST rendering errors. This patch corrects this by injecting empty lines between the changelog entry bodies and their reference lists.

In corner cases, changelog fragments with things like detached link
definitions (example: aio-libs#7346) cause RST rendering errors. This patch
corrects this by injecting empty lines between the changelog entry
bodies and their reference lists.
@webknjaz webknjaz added bot:chronographer:skip This PR does not need to include a change note backport-3.8 labels Jul 8, 2023
@webknjaz webknjaz requested a review from asvetlov as a code owner July 8, 2023 07:36
@webknjaz webknjaz enabled auto-merge (squash) July 8, 2023 07:37
@codecov
Copy link

codecov bot commented Jul 8, 2023

Codecov Report

Merging #7348 (9cfdf35) into master (fb7a0a1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7348   +/-   ##
=======================================
  Coverage   97.28%   97.28%           
=======================================
  Files         106      106           
  Lines       31399    31399           
  Branches     3924     3924           
=======================================
  Hits        30547    30547           
  Misses        648      648           
  Partials      204      204           
Flag Coverage Δ
CI-GHA 97.23% <ø> (ø)
OS-Linux 96.90% <ø> (ø)
OS-Windows 95.36% <ø> (ø)
OS-macOS 96.59% <ø> (ø)
Py-3.10.11 95.28% <ø> (ø)
Py-3.10.12 96.79% <ø> (ø)
Py-3.11.0 96.54% <ø> (ø)
Py-3.8.10 95.25% <ø> (ø)
Py-3.8.17 96.72% <ø> (ø)
Py-3.9.13 95.25% <ø> (ø)
Py-3.9.17 96.75% <ø> (ø)
Py-pypy7.3.11 94.24% <ø> (+<0.01%) ⬆️
VM-macos 96.59% <ø> (ø)
VM-ubuntu 96.90% <ø> (ø)
VM-windows 95.36% <ø> (ø)

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

@webknjaz webknjaz merged commit bf9d753 into aio-libs:master Jul 8, 2023
@patchback
Copy link
Contributor

patchback bot commented Jul 8, 2023

Backport to 3.8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.8/bf9d753edc928e7ecbc590c32603ebd3c1fc6282/pr-7348

Backported as #7349

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@patchback
Copy link
Contributor

patchback bot commented Jul 8, 2023

Backport to 3.9: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.9/bf9d753edc928e7ecbc590c32603ebd3c1fc6282/pr-7348

Backported as #7350

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Jul 8, 2023
webknjaz added a commit that referenced this pull request Jul 8, 2023
…ore links (#7350)

**This is a backport of PR #7348 as merged into master
(bf9d753).**

In corner cases, changelog fragments with things like detached link
definitions (example: #7346) cause RST rendering errors. This patch
corrects this by injecting empty lines between the changelog entry
bodies and their reference lists.

Co-authored-by: Sviatoslav Sydorenko <[email protected]>
webknjaz added a commit that referenced this pull request Jul 8, 2023
…ore links (#7349)

**This is a backport of PR #7348 as merged into master
(bf9d753).**

In corner cases, changelog fragments with things like detached link
definitions (example: #7346) cause RST rendering errors. This patch
corrects this by injecting empty lines between the changelog entry
bodies and their reference lists.

Co-authored-by: Sviatoslav Sydorenko <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:skip This PR does not need to include a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant