EditorConfig: set indent_size = tab
#1949
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Despite
indent_size
being set totab
by default, it turns out that we setindent_size
to2
for*
at the top of this file. So, for everywhere else, explicitly setindent_size
to the default (tab
). This should achieve the goal of my last patch to.editorconfig
.Motivation and Context
Linters and formatters are good; they make tedium easy. But, they need to be configured properly and it was my PR that removed the explicit
indent_size=4
.This PR makes no formatting changes; just fixes my previous patch.
References
EditorConfig.org:
How Has This Been Tested?
Testing? Where we're going, we don't need testing!
Types of changes
Checklist:
clean_files.txt
and formatted it usinglint_clean_files.sh
.