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

Switch curly and no-case-declarations to errors #20

Merged
merged 1 commit into from Sep 11, 2018

Conversation

existentialism
Copy link
Member

@hzoo
Copy link
Member

hzoo commented Mar 3, 2017

btw we have 3 other rules on in master, should get in as many as possible before next release

@loganfsmyth
Copy link
Member

Nice.

Personally, my vote would be to add these to https://github.com/babel/babel/blob/7.0/.eslintrc in a PR alongside the style fixes, then once we're happy, make a PR to this repo. It seems like a pain to slowly accumulate these over here one at a time.

@hzoo
Copy link
Member

hzoo commented Mar 3, 2017

Great idea Logan

@existentialism
Copy link
Member Author

Guess it's time to update this :)

We've been running curly and no-case-declarations on babel/babel for a while!

@existentialism existentialism changed the title Enable curly for multi-line Switch curly and no-case-declarations to errors Sep 11, 2018
@existentialism existentialism merged commit dbf2d4b into master Sep 11, 2018
@existentialism existentialism deleted the existentialism-patch-1 branch September 11, 2018 15:28
nicolo-ribaudo pushed a commit to babel/babel that referenced this pull request Nov 14, 2019
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

Successfully merging this pull request may close these issues.

None yet

3 participants