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

Added editorconfig for easy style conformance. #520

Merged
merged 1 commit into from
Dec 2, 2015

Conversation

mbrukman
Copy link
Contributor

@mbrukman mbrukman commented Dec 2, 2015

More info at http://editorconfig.org - this makes it easy for editors to
automatically perform the right indentation.

@busbey
Copy link
Collaborator

busbey commented Dec 2, 2015

could you update the commit message to include that this is just changing our checkstyle?

e.g.

[checkstyle] added editor config for easy style conformance.

More info at http://editorconfig.org - this makes it easy for editors to
automatically perform the right indentation.

More info at http://editorconfig.org - this makes it easy for editors to
automatically perform the right indentation.
@mbrukman
Copy link
Contributor Author

mbrukman commented Dec 2, 2015

@busbey - done.

busbey added a commit that referenced this pull request Dec 2, 2015
Added editorconfig for easy style conformance.
@busbey busbey merged commit ac4c0a7 into brianfrankcooper:master Dec 2, 2015
@busbey
Copy link
Collaborator

busbey commented Dec 2, 2015

thanks for the contribution!

@mbrukman mbrukman deleted the editorconfig branch December 2, 2015 21:54
@bigbes bigbes mentioned this pull request Dec 20, 2015
jaricftw pushed a commit to jaricftw/YCSB that referenced this pull request Jul 19, 2016
Added editorconfig for easy style conformance.
jaricftw pushed a commit to jaricftw/YCSB that referenced this pull request Jul 19, 2016
Added editorconfig for easy style conformance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants