Skip to content

ci: move job to deploy doc preview to a separte workflow#3627

Merged
ianna merged 9 commits intoscikit-hep:mainfrom
ariostas:workflow_run_docs_preview
Aug 26, 2025
Merged

ci: move job to deploy doc preview to a separte workflow#3627
ianna merged 9 commits intoscikit-hep:mainfrom
ariostas:workflow_run_docs_preview

Conversation

@ariostas
Copy link
Copy Markdown
Member

This PR works alongside #3625 to be able to deploy doc previews for PRs from forks.

This PR needs to be merged first, and then #3625 right after.

I tested this in https://github.com/ariostas-talks/awkward-test/pull/1 to verify that it works properly.

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.48%. Comparing base (b749e49) to head (db9de0e).
⚠️ Report is 395 commits behind head on main.

Additional details and impacted files

see 196 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ariostas
Copy link
Copy Markdown
Member Author

Actually, I first need to figure out how to get this to not trigger on pushes to main

@ariostas
Copy link
Copy Markdown
Member Author

Okay, this is ready now

@ianna ianna added the pr-next-release Required for the next release label Aug 21, 2025
@ianna ianna self-requested a review August 21, 2025 15:10
Copy link
Copy Markdown
Member

@ianna ianna left a comment

Choose a reason for hiding this comment

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

@ariostas - Great! Thanks!

@ianna ianna merged commit efa1466 into scikit-hep:main Aug 26, 2025
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-next-release Required for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants