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

Renormalize indentation whitespace to tabs #368

Merged
merged 1 commit into from
Jun 7, 2017
Merged

Conversation

stakx
Copy link
Contributor

@stakx stakx commented Jun 7, 2017

This fixes #364.

@stakx stakx merged commit c6a2297 into devlooped:develop Jun 7, 2017
@JohanLarsson
Copy link
Contributor

FWIW I prefer four spaces, renders nicer when reading code in the browser if nothing else.

@stakx
Copy link
Contributor Author

stakx commented Jun 7, 2017

Yes, I personally prefer four spaces, too. They have the additional advantage that multi-line (wrapped) statements can be properly indented without running into the question of whether & how to mix tabs and spaces.

For now, in this commit, I did not want to let my personal preferences intrude: I decided to adhere to .editorconfig and to what the majority of Moq's lines of code already use. That doesn't mean that we can't change everything to spaces at a later point in time. At least for now, we have some consistency again.

@stakx stakx deleted the develop branch June 7, 2017 10:47
@devlooped devlooped locked and limited conversation to collaborators Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants