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

Different indentation #35

Closed
farmdawgnation opened this issue Mar 5, 2016 · 4 comments
Closed

Different indentation #35

farmdawgnation opened this issue Mar 5, 2016 · 4 comments
Labels
Milestone

Comments

@farmdawgnation
Copy link
Contributor

There are a few different indentation strategies used throughout the source. Some things are using tabs, others are using spaced. @cvogt Which is intended? :)

@cvogt
Copy link
Owner

cvogt commented Mar 6, 2016

@farmdawgnation there are tabs in there? Not indented. In support for cleaning up indentation in general. Let's coordinate to avoid merge conflicts. Also in favour of Scalariform, if we find a nice way to integrate it, maybe a cbt build ;)

@cvogt
Copy link
Owner

cvogt commented Mar 6, 2016

these are my preferred settings
.setPreference(AlignParameters, true)
.setPreference(AlignArguments, true)
.setPreference(AlignSingleLineCaseStatements, true)
.setPreference(MultilineScaladocCommentsStartOnFirstLine, true)
.setPreference(SpaceInsideParentheses, true)
.setPreference(SpacesWithinPatternBinders, true)
.setPreference(SpacesAroundMultiImports, true)
.setPreference(DoubleIndentClassDeclaration, false)

@cvogt cvogt modified the milestone: 1.0 RC1 (Scala Days) Mar 6, 2016
@cvogt cvogt added the feature label Mar 6, 2016
@farmdawgnation
Copy link
Contributor Author

I'm down for scalariform. Though that won't help tabs in shell scripts and java files. ;)

@cvogt
Copy link
Owner

cvogt commented Mar 7, 2016

so the tabs in Java have been replaced by spaces. I am closing this.

@cvogt cvogt closed this as completed Mar 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants
@cvogt @farmdawgnation and others