-
Notifications
You must be signed in to change notification settings - Fork 2
Get *.js indentation from .editorconfig file #88
Conversation
Semver is not optional. if you make a breaking change you got 1.0. In fact, the fact that you have any project that is <1.0 is silly, all modules should start at v1 or v2 out of the gate. If you have some sort of emotional attachment to version "1.0" then forget that. Communicating with your users is always more important. |
@@ -1,39 +1,12 @@ | |||
'use strict'; | |||
var detectIndent = require('detect-indent'); | |||
require('array.prototype.find'); |
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.
dont do that please. there is a nice var find = require('find')
module.
Please ship with a default editorconfig in the rc folder. |
Got it. Todo:
|
if (parseErr) { | ||
return callback(parseErr); | ||
} | ||
var jsRules = parsed.find(function getJsRuleSet(ruleset) { |
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.
You can just do [].filter()
or [].some()
i think...
Let me know if this is okay now. |
Since I'm going to cut a 1.0.0 for this, I'm also resolving these issues: |
Resolving #76 |
lgtm. |
Fixes: #39
Once this is accepted, I'll cut 0.5.0 (breaking api change since you need an .editorconfig file). Let me know if I should make editorconfig optional. My gut is no since .editorconfig does a lot to help multiple people working on the same project.
cc: @Raynos