Skip to content

bpo-39334: FIX: Deprecated sphinx index notations.#18004

Closed
JulienPalard wants to merge 1 commit intopython:mainfrom
JulienPalard:sphinx-index
Closed

bpo-39334: FIX: Deprecated sphinx index notations.#18004
JulienPalard wants to merge 1 commit intopython:mainfrom
JulienPalard:sphinx-index

Conversation

@JulienPalard
Copy link
Copy Markdown
Member

@JulienPalard JulienPalard commented Jan 14, 2020

I used:

sed -i 's/^\([[:space:]]*\)\.\. index:: \(module\|keyword\|operator\|object\|exception\|statement\|builtin\): \(.*\)$/\1.. index:: pair: \2; \3/' Doc/**/*.rst
sed -i '/^[[:space:]]*\.\.[[:space:]]index::/,/^$/{s/^\([[:space:]]\+\)\(! \)\?\(module\|keyword\|operator\|object\|exception\|statement\|builtin\): \(.*\)$/\1pair: \3; \2\4/}' Doc/**/*.rst

see: sphinx-doc/sphinx#6970

https://bugs.python.org/issue39334

I used:

sed -i 's/^\([[:space:]]*\)\.\. index:: \(module\|keyword\|operator\|object\|exception\|statement\|builtin\): \(.*\)$/\1.. index:: pair: \2; \3/' Doc/**/*.rst
sed -i '/^[[:space:]]*\.\.[[:space:]]index::/,/^$/{s/^\([[:space:]]\+\)\(! \)\?\(module\|keyword\|operator\|object\|exception\|statement\|builtin\): \(.*\)$/\1pair: \3; \2\4/}' Doc/**/*.rst
@JulienPalard JulienPalard requested a review from tiran as a code owner January 14, 2020 20:50
@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting core review labels Jan 14, 2020
@JulienPalard JulienPalard changed the title FIX: Deprecated sphinx index notations. bpo-39334: FIX: Deprecated sphinx index notations. Jan 14, 2020
@csabella
Copy link
Copy Markdown
Contributor

I added the DO NOT MERGE label as the upstream change was withdrawn. I'm not sure if this is a change that should be considered anyway or if Serhiy's comment on the bug tracker should be applied instead.

@tiran tiran removed their request for review April 18, 2021 09:42
@JulienPalard
Copy link
Copy Markdown
Member Author

As sphinx-doc/sphinx#6970 seems stuck I'm closing this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants