-
Notifications
You must be signed in to change notification settings - Fork 10
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
Support for --all-namespaces #3
Comments
Adding an |
Agreed, this would then decouple the Github actions needing to maintain new inputs as the upstream CLI added new functionality. I also noticed here we're pinning the image to use |
Alas you can't (or at least couldn't) pass a variable from the Action to the image. If you've seen a nice pattern for that I'd definitely be interested. |
@garethr correct the way it has been implemented does make it difficult. I helped do something similar for the It wasn't exactly how I'd like but they now use The alternative is to explicitly use the image:tag with the version then release the action with tags so you can reference the action like following:
I think either approach is fine, I find the latter requires more maintenance on this repo to ensure the new releases are maintained and tagged so the first option being able to parse the version puts that effort into the users' hands. |
conftest-action/action.yml
Line 29 in ae9212d
I've like to be able to specify any number of the supported flags available in the conftest CLI can we have an input that allows me to specify my own flags? Or can we get support for
--all-namespaces
in the interim?The text was updated successfully, but these errors were encountered: