Bump checkout version #1369

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

vloncar merged 12 commits into fastmachinelearning:main from JanFSchulte:checkout_bump

Oct 30, 2025

Contributor

JanFSchulte commented Aug 21, 2025

#1367 seems to break the mirroring of the repo to gitlab. The bump to the version of the checkout action are still necessary, so this PR is meant to replace the broken one.

Type of change

Other (Specify)

Tests

Checklist

N/A


          bump checkout version

332be48

JanFSchulte mentioned this pull request

Bump actions/checkout from 4 to 5 #1367

Closed

jmitrevs added the please test label

Contributor

jmitrevs commented Aug 22, 2025

Do you understand the test failure?

Contributor

vloncar commented Aug 22, 2025

We need an extra permission for the sphinx action. I can get to it once I'm near a computer.

JanFSchulte added 2 commits

October 30, 2025 13:43


          update sphinx-build workflow to use new fine-graned PAT

1948a11


          Merge branch 'main' into checkout_bump

cd576fd

JanFSchulte added please test and removed please test labels

JanFSchulte added 2 commits

October 30, 2025 13:52


          token with workflow permissions

51c75f1


          Merge branch 'checkout_bump' of https://github.com/JanFSchulte/hls4ml …

8ccd05d

…into checkout_bump

JanFSchulte added please test and removed please test labels


          try to give permissions to change workflow files

4f522b0

JanFSchulte added github_actions please test and removed please test labels


          try to give permissions to change workflow files 2

143af35

JanFSchulte added please test and removed please test github_actions labels


          try to give permissions to change workflow files 3

6bf20d6

JanFSchulte added please test and removed please test labels


          try to give permissions to change workflow files 4

0ae4c21

JanFSchulte added please test and removed please test labels

JanFSchulte added the please test label


          try to force token

ceb0207

JanFSchulte added please test and removed please test labels


          check git config

ccdab9e

JanFSchulte added please test and removed please test labels


          use 2 differnet tokens with appropriate permissions and clean up debu…

073ec59

…gging stuff

JanFSchulte added please test and removed please test labels

JanFSchulte requested review from jmduarte and vloncar

October 30, 2025 19:58

Contributor Author

JanFSchulte commented Oct 30, 2025

I have set up fresh tokens for the repo that we need to keep the actions working after this update. It works nicely for the test-sphinx/build one, but the update of the branch on PR from fork is not able to be tested in this PR and might still be problematic. I think it should work, but we probably just need to bite the bullet, merge this, and then see if everything works.

vloncar approved these changes

View reviewed changes

vloncar merged commit a209862 into fastmachinelearning:main

4 of 6 checks passed

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

Labels