chore: set permission levels to read and pin specific commit hashes#681
chore: set permission levels to read and pin specific commit hashes#681abelaba wants to merge 13 commits intooptimagic-dev:mainfrom
Conversation
c396a42 to
7523062
Compare
|
@AdrianoKF could you review my last commit? I split the build-and-publish workflow that was previously combined into separate steps, so the permission |
|
In general this looks good to me! A few small observations:
|
|
@abelaba - if you need help in setting up Trusted Publishing, just ping me! After the first release, let's also validate that attestations are correctly uploaded to PyPI for the release files. |
|
@AdrianoKF Thank you for the review. I used |
This reverts commit 0c07aea.
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
New changes
|
Summary
contents: readfor main.yml workflow.pinact.persist-credentials: falseto all actions/checkout steps to avoid exposing theGITHUB_TOKENto subsequent steps.CODECOV_TOKENis passed through a GitHub environment, and not shared across jobs.codecovafter all tests have finished running in this jobrun-tests-linux.Action Items before merging
publish-to-pypi.yml.codecov. Environment name I set for the job.CODECOV_TOKEN.