Skip to content

Infra: replace deprecated action with RtD app#4946

Merged
willingc merged 1 commit intopython:mainfrom
hugovk:documentation-links.yml
Apr 28, 2026
Merged

Infra: replace deprecated action with RtD app#4946
willingc merged 1 commit intopython:mainfrom
hugovk:documentation-links.yml

Conversation

@hugovk
Copy link
Copy Markdown
Member

@hugovk hugovk commented Apr 28, 2026

For python/core-workflow#587.

The readthedocs/actions/preview is deprecated, and uses the pull_request_target target which can be insecure.

Merging this means we don't get the edited PR messages with a preview link. Let's replace it with the Read the Docs app, which leaves a comment with the preview.


📚 Documentation preview 📚: https://pep-previews--4946.org.readthedocs.build/

@hugovk hugovk added the infra Core infrastructure for building and rendering PEPs label Apr 28, 2026
@read-the-docs-community
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

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

Thanks @hugovk.

@willingc willingc merged commit 0b2298a into python:main Apr 28, 2026
6 checks passed
@hugovk hugovk deleted the documentation-links.yml branch April 29, 2026 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Core infrastructure for building and rendering PEPs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants