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

More white space please (padded-blocks) #11

Closed
ba55ie opened this issue Sep 30, 2016 · 3 comments
Closed

More white space please (padded-blocks) #11

ba55ie opened this issue Sep 30, 2016 · 3 comments
Labels

Comments

@ba55ie
Copy link
Contributor

ba55ie commented Sep 30, 2016

Enable padded-blocks for at least Classes.

We really would like to have something like: ['error', 'always' { blocks: 'never', switches: 'never'}] which implicitly means: classes: 'always' if I stand correctly?

@ba55ie
Copy link
Contributor Author

ba55ie commented Oct 6, 2016

It's more or less all or nothing for now. We could change it to: 'padded-blocks': ['error', { blocks: 'never', switches: 'never', classes: 'never' }], but that seems inconsistent.

There are many requests for more configuration options, so on-hold untill for now.

eslint/eslint#3092
eslint/eslint#5949
eslint/eslint#7145

@ba55ie
Copy link
Contributor Author

ba55ie commented May 23, 2017

Seems like it will be in V4: http://eslint.org/docs/4.0.0/rules/padding-line-between-statements

@ba55ie
Copy link
Contributor Author

ba55ie commented Oct 17, 2017

Fixed by this new rule: https://eslint.org/docs/rules/lines-between-class-members

@ba55ie ba55ie closed this as completed Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant