-
Notifications
You must be signed in to change notification settings - Fork 264
Added .editorconfig file #877
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
Conversation
It's kinda silly to do a direct copy from a different project with different types of files and a different directory structure. Half of these rules are for files that don't even exist, and the majority of the files in this repo don't even get indentation rules applied to them. |
For the benefit of solving this issue, it'd be great if you could point on specific lines that you'd like to change/add/remove. By doing so, we can have a change that works for everyone and improve the status quo. In its current state your comment doesn't really help. |
@AurelioDeRosa: why bother specifying |
Right, almost everything is tab indented, so having that be the default would make sense. |
trim_trailing_whitespace = true | ||
insert_final_newline = true | ||
|
||
# Tabs in JS unless otherwise specified |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is nothing else specified.
@AurelioDeRosa It seems to me that current version of Core's Would you like to do it? If not, I'd close this PR and I can do it separately. |
@AurelioDeRosa ping |
.editorconfig
file taken directly from jQuery's repository.