You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With input configfile one can specify the configuration file (defaults to fix-whitespace.yaml).
I find the default confusing in a scenario when I try to repro it locally and I can't run the action. I run fix-whitespace manually, and, of course, I get different results if I forget that I need to specify the config manually.
In short, I think the action and the tool should have the same default, if possible. Perhaps, people will be unhappy if you remove the default value from the action. So, a less disruptive way would be to add the same default to the tool.
The text was updated successfully, but these errors were encountered:
I find the default confusing in a scenario when I try to repro it locally and I can't run the action. I run
fix-whitespace
manually, and, of course, I get different results if I forget that I need to specify the config manually.In short, I think the action and the tool should have the same default, if possible. Perhaps, people will be unhappy if you remove the default value from the action. So, a less disruptive way would be to add the same default to the tool.
The text was updated successfully, but these errors were encountered: