-
Notifications
You must be signed in to change notification settings - Fork 13
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
Various fixes #243
Various fixes #243
Conversation
Looking at the change for the sh file, how about we run shellcheck on all of them? |
@HidenoriKobayashi enabled |
@HidenoriKobayashi Please take a look when you have a time. Thank you! |
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. Thanks for integrating the shellcheck check.
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.
For the cro3.bash
part looks good to me.
Thanks!
List of the changes: