Skip to content
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

Fix the indentation to 2 spaces #107

Merged
merged 2 commits into from Oct 31, 2017

Conversation

@Shwetajain148
Copy link
Contributor

@Shwetajain148 Shwetajain148 commented Oct 13, 2017

I have fixed the indentation of the scripts to 2 spaces.

Kindly review and merge.

@Shwetajain148 Shwetajain148 force-pushed the Shwetajain148:Fix-Indentation-To-2spaces branch from e7f3d2b to 263abc2 Oct 13, 2017
@mostlyjason
Copy link
Contributor

@mostlyjason mostlyjason commented Oct 17, 2017

Looks fine to me. @mchaudhary how do you think the 2 spaces look? could also see 4 spaces working, but 8 was too many.

@Shwetajain148 Shwetajain148 force-pushed the Shwetajain148:Fix-Indentation-To-2spaces branch from 927074f to 1289784 Oct 24, 2017
@Shwetajain148 Shwetajain148 force-pushed the Shwetajain148:Fix-Indentation-To-2spaces branch from 1289784 to 80b404c Oct 24, 2017
@mchaudhary mchaudhary merged commit 3757ab3 into loggly:master Oct 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.