-
Notifications
You must be signed in to change notification settings - Fork 29k
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
fs: remove unnecessary option argument validation #53958
Conversation
6327d7b
to
f4e9f08
Compare
CI is failing |
I guess we are also testing the execution order here. We should update the tests @JonasBa |
Yeah, sorry about that. My device is managed and I cant change firewall rules to run the tests without getting a million connection popups |
You can run "tools/test.py path/to/test/file" for specific tests |
20a13bf
to
89a14bf
Compare
@anonrig is ti common for the build to run so long or did this timeout and needs a rerun? I dont see any failures, but I lack context on being able to tell how long this should take :) |
This is a timeout probably |
Landed in 3ed9f98 |
Thank you @anonrig 🙏🏼 |
PR-URL: #53958 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Remove unnecessary option parsing in mkdir