-
Notifications
You must be signed in to change notification settings - Fork 158
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
feature: warn when reusable workflow is used as a step #454
base: main
Are you sure you want to change the base?
Conversation
I'm not sure what this feature tries to solve. Is it a common mistake to pass a workflow YAML file to |
@rhysd I'm not sure how common that is, however, Github currently does not warn if someone does that. |
For context, there's examples of people asking about this: It's not common, though 🙂 |
Okay, I'm not against this feature but I'd like to make false positives as minimal as possible. One possible false positive is local actions under - uses: ./.github/workflows/actions/my-action
with:
arg: ... This check will report the valid action usage as invalid reusable workflow usage. |
Sure, I will fix that and try to test as much cases as possible |
05e056b
to
5aaa4ce
Compare
Using reusable workflow as a step is not supported by Github Actions.
However, actionlint currently does not warn about this behavior.
Now it does 🙂