Skip to content

[doc] Fix dis markup#23524

Merged
gpshead merged 1 commit intopython:masterfrom
andresdelfino:dis_markup
Nov 28, 2020
Merged

[doc] Fix dis markup#23524
gpshead merged 1 commit intopython:masterfrom
andresdelfino:dis_markup

Conversation

@andresdelfino
Copy link
Contributor

@andresdelfino andresdelfino commented Nov 26, 2020

No description provided.

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Nov 26, 2020
@gpshead gpshead merged commit fa840cc into python:master Nov 28, 2020
@miss-islington
Copy link
Contributor

miss-islington commented Nov 28, 2020

Thanks @andresdelfino for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@gpshead
Copy link
Member

gpshead commented Nov 28, 2020

thanks!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 28, 2020
@andresdelfino @miss-islington
Fix dis markup (pythonGH-23524)
4b482f4 
(cherry picked from commit fa840cc)

Co-authored-by: Andre Delfino <[email protected]>
@bedevere-bot
Copy link

bedevere-bot commented Nov 28, 2020

GH-23545 is a backport of this pull request to the 3.9 branch.

miss-islington added a commit that referenced this pull request Nov 28, 2020
@miss-islington @andresdelfino
Fix dis markup (GH-23524)
cfb2f79 
(cherry picked from commit fa840cc)

Co-authored-by: Andre Delfino <[email protected]>
@andresdelfino andresdelfino deleted the dis_markup branch November 28, 2020 22:27
@rhettinger
Copy link
Contributor

rhettinger commented Nov 29, 2020

Does this have any visual effect for the reader?

@serhiy-storchaka
Copy link
Member

serhiy-storchaka commented Nov 29, 2020

Good catch! And it is common error. Perhaps we can add a check in Doc/tools/rstlint.py which checks that the text following a directive does not have excessive indentation.

@mdickinson
Copy link
Member

mdickinson commented Nov 29, 2020

which checks that the text following a directive does not have excessive indentation

I think the issue wasn't excessive indentation, but instead inconsistent indentation within the block for the directive. The text was indented four spaces while the .. versionadded directive that was also part of the same block was indented three spaces. Indenting the .. versionadded directive one extra space would also have fixed the issue.

@andresdelfino
Copy link
Contributor Author

andresdelfino commented Nov 29, 2020

Does this have any visual effect for the reader?

Yes.

adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
@andresdelfino @adorilson
Fix dis markup (pythonGH-23524)
1861071
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gpshead gpshead gpshead approved these changes

@Fidget-Spinner Fidget-Spinner Fidget-Spinner approved these changes

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@andresdelfino @miss-islington @gpshead @bedevere-bot @rhettinger @serhiy-storchaka @mdickinson @Fidget-Spinner @the-knights-who-say-ni