Skip to content

ci: fix checkout action parameter - #1250

Closed
wagner-intevation wants to merge 2 commits into
CERTCC:mainfrom
wagner-intevation:fix/1226
Closed

wagner-intevation wants to merge 2 commits into
CERTCC:mainfrom
wagner-intevation:fix/1226

Conversation

@wagner-intevation

Copy link
Copy Markdown

as required by the doctools action
@wagner-intevation

Copy link
Copy Markdown
Author

The suggestion in #1247 (comment) is unnecessarily complex. AFAIK, actions/checkout with default parameters works fine.

@sei-vsarvepalli

Copy link
Copy Markdown
Contributor

This still fails - see #1250 where there is the fix, we ignore generatedAt irrespective of which file has it. Whenever we touch these workflow files, we also want to update them to pin to SHA256sum in the action: attribute in yaml.

@wagner-intevation

wagner-intevation commented Sep 17, 2026

Copy link
Copy Markdown
Author

This still fails - see #1250

If you mean this run: https://github.com/CERTCC/SSVC/actions/runs/35239830122/job/105265173370#step:6:21
The checkout succeeded, but it failed at the unrelated step Fail if fork PR has uncommitted data/ changes.

I then did what the command output told me to do - run make regenerate_json, which deletes lots of data. Looks strange to me, but idk.

@sei-vsarvepalli

Copy link
Copy Markdown
Contributor

yeh that failure is due to the generatedAt that you mentioned in the earlier comment. That was helpful in me fine tuning it. I can close on this one for now, as duplicate. @bernhardreiter has a few more assigned to him for help.

Duplicate to #1249 - resolving 1233

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New PR causes the git fetch to fail

2 participants