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

Add builtin rules for whitespaced code #71

Open
nervetattoo opened this issue Apr 3, 2013 · 2 comments
Open

Add builtin rules for whitespaced code #71

nervetattoo opened this issue Apr 3, 2013 · 2 comments

Comments

@nervetattoo
Copy link
Member

I think it makes sense to ship builtin rules for both tab and whitespace indentation.
Too many projects will be using different rules here so it doesn't make sense that every project needs to write custom code just to enforce their standard.

I suggest we rename the HasTabsAsIndentation to HasCorrectIndentation and passes the actual rule as a configuration:

{
    "variables": {
        "HasCorrectIndentation": {
            "character": "\t",
            "num": 1
        }
    }
}

I'll create a PR if theres no disagreement to this.

@jails
Copy link
Contributor

jails commented Apr 13, 2013

Yeah pretty agree for merging HasCorrectTabIndention and HasTabsAsIndentation to a configurable HasCorrectIndentation rule.

@nateabele
Copy link
Member

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants