-
Notifications
You must be signed in to change notification settings - Fork 128
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
[bug] Fix pre-submit #3320
Comments
This could get tricky to plan the arguments to I guess it could work like this, to only use default for the
|
To be fair, this is the case for all of our actions and workflows when run in pre-submits. We've generally solved this with scheduled e2e tests. |
Another way to solve this is to have a test that calls the wrapped action directly. This doesn't ensure that everything works for the action end to end but I can ensure that the changed code is executed at least. |
See #3312
The secure-upload-folder Action is broken and always runs at main instead of using the PR code.
I think we can solve this by doing:
/cc @ramonpetgrave64
The text was updated successfully, but these errors were encountered: