Skip to content

fix: adding generated action tests#181

Merged
dotkas merged 1 commit intomasterfrom
dotkas/add-generated-action-tests
Aug 15, 2025
Merged

fix: adding generated action tests#181
dotkas merged 1 commit intomasterfrom
dotkas/add-generated-action-tests

Conversation

@dotkas
Copy link
Copy Markdown
Contributor

@dotkas dotkas commented Aug 13, 2025

Ensure we're not adding actions for which Snyk is not installed or where an image does not exist.

@dotkas dotkas requested review from a team as code owners August 13, 2025 11:12
@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Aug 13, 2025

🎉 Snyk checks have passed. No issues have been found so far.

code/snyk check is complete. No issues have been found. (View Details)

@dotkas dotkas force-pushed the dotkas/add-generated-action-tests branch 10 times, most recently from f082dad to ccfd489 Compare August 13, 2025 16:40
@dotkas dotkas changed the title fix: adding generated action steps fix: adding generated action tests Aug 13, 2025
@dotkas dotkas force-pushed the dotkas/add-generated-action-tests branch 3 times, most recently from 1fc50d1 to 332aa64 Compare August 14, 2025 16:50
Comment on lines +50 to +53
- uses: actions/checkout@v4
# This looks odd, but the `matrix.variant` as input in `uses` is not supported by Github.
# Therefore, we must add every step as a conditional instead, if we don't want to generate an
# individual workflow for each variant.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

As the comment states, I was unable to find a way to get GH actions to accept matrix.variant as an input variable to uses if the runner was docker. Hence this oddity.

@dotkas dotkas enabled auto-merge (squash) August 14, 2025 17:42
# Conflicts:
#	README.md
#	_templates/BASE.md.erb
@dotkas dotkas force-pushed the dotkas/add-generated-action-tests branch from 332aa64 to 18ea261 Compare August 15, 2025 11:04
@dotkas dotkas merged commit c4a3328 into master Aug 15, 2025
25 of 26 checks passed
@dotkas dotkas deleted the dotkas/add-generated-action-tests branch August 15, 2025 11:05
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.

2 participants