-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add bashate to CI #417
Add bashate to CI #417
Conversation
@yegor256 Quoting |
@markparonyan you should put all names into an array and then use the array as Just imagine, what will happen if you do this |
@yegor256 Thanks! I fixed that. Should I also refactor bash scripts across the project according to bashate? |
@markparonyan yes, of course you have to fix all bashate warnings before we can merge. Other issues -- make a different ticket about them, we'll see what we can do. |
@yegor256 please check |
@markparonyan thanks a lot! |
ref #411