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

Update to eslint-config-hapi@2.x.x #436

Merged
merged 1 commit into from Sep 11, 2015
Merged

Update to eslint-config-hapi@2.x.x #436

merged 1 commit into from Sep 11, 2015

Conversation

@cjihrig
Copy link
Contributor

cjihrig commented Sep 9, 2015

This release contains proper linting for switch-case statements (with tests to help prevent regressions). I bumped the major version because existing lint jobs that depended on the old behavior would begin failing. I'll leave it up to you to sort out how you want to version lab. Verified that this closes #434.

UPDATE: Released v2.0.1. Closes #437

@hueniverse

This comment has been minimized.

Copy link
Member

hueniverse commented Sep 9, 2015

I think the prev rule was a bug and it actual broke hapi code so I vote for patch.

@cjihrig cjihrig mentioned this pull request Sep 10, 2015
@geek geek added this to the 5.16.1 milestone Sep 11, 2015
@geek geek added the dependency label Sep 11, 2015
@geek geek self-assigned this Sep 11, 2015
geek added a commit that referenced this pull request Sep 11, 2015
Update to eslint-config-hapi@2.x.x
@geek geek merged commit df2ae8d into hapijs:master Sep 11, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@cjihrig cjihrig deleted the cjihrig:eslint-config branch Sep 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.