Skip to content
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.

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

Add support for Action manifest called action.yaml #19

Merged
merged 1 commit into from
Aug 25, 2023

Conversation

ericcornelissen
Copy link
Owner

Closes #16

Summary

Extend support for manifest scanning to support both the .yml and .yaml extensions per: https://docs.github.com/en/actions/creating-actions/about-custom-actions. CLI tests have been extended to cover .yaml extension manifest as well - both for directory and file scanning.

Extend support for manifest scanning to support both the `.yml` and
`.yaml` extensions per:
https://docs.github.com/en/actions/creating-actions/about-custom-actions
CLI tests have been extended to cover `.yaml` extension manifest as well
- both for directory and file scanning.

A direct implementation (i.e. without ` || hasProblems`) caused existing
tests to faill due to an `action.yaml` manifest resetting `hasProblems`
for an `action.yml` manifest. As a general precaution, any assignment to
`hasProblems` now ORs it with it's existing value to avoid resets.

Signed-off-by: Eric Cornelissen <ericornelissen@gmail.com>
@ericcornelissen ericcornelissen added the bug Something isn't working label Aug 25, 2023
@ericcornelissen ericcornelissen merged commit 92f7838 into main Aug 25, 2023
9 checks passed
@ericcornelissen ericcornelissen deleted the action-yaml-manifest branch August 25, 2023 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

action.yaml is a valid action manifest name
1 participant