FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix wrongly indented nested list in `index.rst`. by ezio-melotti · Pull Request #1184 · python/devguide · GitHub

  1. python
  2. devguide

Repository navigation

Fix wrongly indented nested list in index.rst. - #1184

Merged
ezio-melotti merged 1 commit into
python:mainfrom
ezio-melotti:index-list-bugs
Oct 10, 2023
ConversationCommits (1)ChecksFiles changed
Merged

Fix wrongly indented nested list in index.rst.#1184
ezio-melotti merged 1 commit into
python:mainfrom
ezio-melotti:index-list-bugs

Conversation

ezio-melotti commented Oct 10, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

Follow-up of #1183 -- fixes a similar error in index.rst.

📚 Documentation preview 📚: https://cpython-devguide--1184.org.readthedocs.build/

ezio-melotti self-assigned this Oct 10, 2023
ezio-melotti mentioned this pull request Oct 10, 2023
Closed
6 tasks done
hugovk reviewed Oct 10, 2023
View reviewed changes
Comment thread index.rst
* :PEP:`7` (Style Guide for C Code)
* :PEP:`8` (Style Guide for Python Code)

* :PEP:`7` (Style Guide for C Code)

hugovk Oct 10, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Should these be three-space indents?

https://devguide.python.org/documentation/style-guide/#use-of-whitespace

ezio-melotti Oct 10, 2023

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Lists work a bit differently (and that's the reason why people often get them wrong). See
https://devguide.python.org/documentation/markup/#lists-and-quotes

We could clarify these exceptions in the devguide (possibly in both places).

pradyunsg Oct 15, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

For posterity: Using three-space indents in this specific spot would result in a blockquote (or dl, depending on the absence of a blank line) in this specific spot.

ezio-melotti merged commit 862e98d into python:main Oct 10, 2023
ezio-melotti deleted the index-list-bugs branch October 10, 2023 15:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

hugovk hugovk approved these changes
AA-Turner AA-Turner approved these changes
pradyunsg Awaiting requested review from pradyunsg

Assignees

ezio-melotti

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL