-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Aliases: lint the entire folder #2047
Conversation
bc4cadc
to
fd79b12
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although this touches 48 files:
- 30 of them are just the file headers,
- 12 of them are just headers and whitespace,
- and the last six are all trivial changes.
fd79b12
to
16a2e27
Compare
9f363ba
to
f8c2149
Compare
c3bad49
to
ecb2437
Compare
there appears to be some failing lint somewhere, I will be happy to merge this after it gets fixed :) |
@NoahGorny, fixed to use single-quotes 👍 |
ba29245
to
f49b0e6
Compare
My apologies to future `git blame` hunters ♥
Alsö, remove impossible alias. If someone wants it, they can write the function, but since aliases literally don't work this way it seems obvious that nobody has ever used it.
f49b0e6
to
12a734c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
shfmt
shellcheck
Motivation and Context
Although this touches 48 files:
How Has This Been Tested?
Types of changes
Checklist:
clean_files.txt
and formatted it usinglint_clean_files.sh
.