-
Notifications
You must be signed in to change notification settings - Fork 198
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
made shell scripts all have 4 spaces for indents not 2; issue #272 #273
Conversation
e43617e
to
9876100
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.
One requested change -- otherwise I think this is OK assuming we want to enforce 4 spaces instead of 2
@HalosGhost is this good to merge? |
Additionally, instead of saying |
20acfa1
to
89200c7
Compare
…t-dci#272 Signed-off-by: Morgan Rockett <morgan.rockett@tufts.edu>
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.
closing the review, thanks
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.
unifies indentation for shell scripts.
Looks good to me.
@maurermi when you have a moment; mind double-checking that there's nothing left that you'd want to see changed? |
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.
ACK, just unifies indentation and makes no significant or concerning changes, looks good to me
Now all shell scripts have indentation at 4 spaces. Previously it was mixed between 2 and 4 spaces for indents.