Skip to content

Add Netlify preview deployments for pull requests#14715

Merged
skjnldsv merged 5 commits intomasterfrom
copilot/analyze-current-workflow-netlify
May 4, 2026
Merged

Add Netlify preview deployments for pull requests#14715
skjnldsv merged 5 commits intomasterfrom
copilot/analyze-current-workflow-netlify

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 2, 2026

🚀 Add Netlify preview deployments for pull requests

Reviewing docs PRs right now basically means trusting that the RST renders fine and the formatting looks okay. Spoiler: it often doesn't, and nobody finds out until after merge.

This adds automatic Netlify preview deployments for every pull request. Each PR gets its own stable URL that updates on every push, so instead of "looks good to me" based on raw RST, reviewers can actually click a link and see the finished result.

All three manuals and PDFs are included. It runs in parallel with the existing checks so there's no extra wait time, and if Netlify has a bad day it won't block the PR either.

💬 Comment example

📖 Documentation Preview

🔍 Open preview →

📄 13 changed documentation pages

Last updated: Sun, 03 May 2026 12:00:00 GMT

@github-actions github-actions Bot added github_actions Pull requests that update GitHub Actions code 3. to review labels May 2, 2026
Copilot AI requested a review from skjnldsv May 2, 2026 11:29
@skjnldsv skjnldsv changed the title Analyzing current workflow and Netlify integration Add Netlify preview deployments for pull requests May 3, 2026
@skjnldsv skjnldsv requested a review from miaulalala May 3, 2026 09:48
@skjnldsv skjnldsv marked this pull request as ready for review May 3, 2026 09:48
Copilot AI requested a review from skjnldsv May 3, 2026 10:00
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 3, 2026

📖 Documentation Preview

🔍 Open preview →

📄 5 changed documentation pages

Last updated: Mon, 04 May 2026 10:15:01 GMT

@skjnldsv skjnldsv force-pushed the copilot/analyze-current-workflow-netlify branch from 0bc00cb to 554fbed Compare May 3, 2026 15:38
@skjnldsv skjnldsv force-pushed the copilot/analyze-current-workflow-netlify branch from d0dde23 to d2b9461 Compare May 3, 2026 16:03
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv force-pushed the copilot/analyze-current-workflow-netlify branch from d2b9461 to 40f57e9 Compare May 3, 2026 16:05
Copy link
Copy Markdown
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

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

Very nice improvement 🎉

Didn't follow the "Post PR preview comment" job in detail, but as it seems to work well (in this PR, without RST changes), it's a good starting point in any case.

@miaulalala
Copy link
Copy Markdown
Contributor

Can we make this an "on demand" feature / so it only runs on bigger changesets? Building for one line changes seems to me to be superfluous.

…-netlify-preview

Docs: small wording and link consistency fixes
@skjnldsv
Copy link
Copy Markdown
Member

skjnldsv commented May 4, 2026

Can we make this an "on demand" feature / so it only runs on bigger changesets? Building for one line changes seems to me to be superfluous.

It shouldn't really matter, were just pushing files that are already built for CI anyway :)
The netlify step is also optional for mergeable state

@skjnldsv skjnldsv merged commit ff19f83 into master May 4, 2026
24 checks passed
@skjnldsv skjnldsv deleted the copilot/analyze-current-workflow-netlify branch May 4, 2026 18:12
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 manual: admin manual: user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants