Skip to content

Fix switch statement indentation rule#606

Merged
ljharb merged 1 commit into
masterfrom
ljharb/fix_switch_indent
Dec 3, 2015
Merged

Fix switch statement indentation rule#606
ljharb merged 1 commit into
masterfrom
ljharb/fix_switch_indent

Conversation

@ljharb
Copy link
Copy Markdown
Collaborator

@ljharb ljharb commented Dec 3, 2015

I abhor switch statements, but their indentation should be:

switch (foo) {
  case 'bar':
    break;
}

not what the plugin requires by default:

switch (foo) {
case: 'bar':
  break;
}

```js
switch (foo) {
  case 'bar':
    break;
}
```

not what the plugin currently requires:
```js
switch (foo) {
case: 'bar':
  break;
}
```
@goatslacker
Copy link
Copy Markdown
Collaborator

👍

ljharb added a commit that referenced this pull request Dec 3, 2015
[breaking] Fix switch statement indentation rule
@ljharb ljharb merged commit 2c3b75d into master Dec 3, 2015
@ljharb ljharb deleted the ljharb/fix_switch_indent branch December 3, 2015 21:16
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.

2 participants