Skip to content

ci: fix deploy job#4082

Merged
ianna merged 1 commit into
scikit-hep:mainfrom
ariostas:fix_deploy_headers
Jun 9, 2026
Merged

ci: fix deploy job#4082
ianna merged 1 commit into
scikit-hep:mainfrom
ariostas:fix_deploy_headers

Conversation

@ariostas

@ariostas ariostas commented Jun 8, 2026

Copy link
Copy Markdown
Member

This PR fixes #4081. It was caused by #4075 where the upload step was changed from a third-party action to just usin gh, which is safer. Since the repository is not checked out, it needed to be specified with --repo.

@codecov

codecov Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@e8bec70). Learn more about missing BASE report.

Additional details and impacted files

@ianna ianna left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ariostas — Great ! Thanks for fixing it.

@ianna
ianna merged commit ebfa120 into scikit-hep:main Jun 9, 2026
17 checks passed
@TaiSakuma TaiSakuma added the type/ci PR title type: ci (set automatically) label Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/ci PR title type: ci (set automatically)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deploy: publish-headers job fails — gh release upload has no repo context (regression from #4075)

3 participants