Skip to content

Replace deprecated action with RtD app#149111

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

Replace deprecated action with RtD app#149111
hugovk merged 1 commit intopython:mainfrom
hugovk:rm-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.

For example: #149109 (comment)

@hugovk hugovk changed the title Replace deprecated workflow with RtD app Replace deprecated action with RtD app Apr 28, 2026
@hugovk hugovk added skip issue skip news infra CI, GitHub Actions, buildbots, Dependabot, etc. labels Apr 28, 2026
@read-the-docs-community
Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #32455606 | 📁 Comparing 5f03495 against main (8a8d737)

  🔍 Preview build  

5 files changed · ± 5 modified

± Modified

@hugovk hugovk enabled auto-merge (squash) April 28, 2026 17:29
@hugovk hugovk merged commit 933e2dd into python:main Apr 28, 2026
60 checks passed
@hugovk hugovk deleted the rm-documentation-links.yml branch April 28, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra CI, GitHub Actions, buildbots, Dependabot, etc. skip issue skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants