-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
ls: silently ignore -T
option
#3718
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
Conversation
i don't think we can mix clap 3.1 & 3.2 |
ls: remove tests for tabsize arg validation
Ok, I reverted the changes and removed the relevant tests. |
many CI jobs are failing, could you please have a look? |
They all say
I don't think I changed anything that would break CI, I just added a Clap arg and a test function. I can replicate the error locally, but the error is in |
Most of the CI jobs magically fixed themselves, and I fixed the style/spelling issues. Should be ready to merge now, unless you have any other issues? |
* ls: silently ignore `-T` option
* ls: silently ignore `-T` option
The option can't be implemented without a significant rewrite according to #3624, and right now passing the option causes a failure.
-T
option is passed-T
option