Skip to content

fix: skip deploy PR when only lastupdated timestamps or epub/pdf changed#15049

Open
Copilot wants to merge 2 commits into
masterfrom
copilot/15045
Open

fix: skip deploy PR when only lastupdated timestamps or epub/pdf changed#15049
Copilot wants to merge 2 commits into
masterfrom
copilot/15045

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 31, 2026

Else we get docs update every day like #15046

Copilot AI added 2 commits May 31, 2026 06:12
When Sphinx rebuilds documentation without any content changes, it still
updates the `<span class="lastupdated">` date in every HTML file and
regenerates epub/pdf binaries. This caused a noisy automated PR (e.g.
#15046) with no meaningful content change.

The `has_changes` check in the deploy job now ignores:
- HTML lines matching `lastupdated` or `Last updated on`
- epub and pdf binary files (which regenerate alongside HTML)

If all changes fall into those categories, no PR is created.
@skjnldsv skjnldsv marked this pull request as ready for review May 31, 2026 06:15
@skjnldsv skjnldsv self-requested a review as a code owner May 31, 2026 06:15
@github-actions github-actions Bot added github_actions Pull requests that update GitHub Actions code 3. to review labels May 31, 2026
@skjnldsv skjnldsv changed the title Adjusting script logic for PR creation fix: skip deploy PR when only lastupdated timestamps or epub/pdf changed May 31, 2026
@skjnldsv skjnldsv requested a review from miaulalala May 31, 2026 06:17
@github-actions
Copy link
Copy Markdown
Contributor

📖 Documentation Preview

🔍 Open preview →

No RST documentation pages changed in this PR.

Last updated: Sun, 31 May 2026 06:20:48 GMT

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

Labels

3. to review github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants