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 eslint and verb dependencies #38

Merged
merged 2 commits into from Sep 17, 2015
Merged

Conversation

anselmbradford
Copy link
Member

Specified verb version was returning error Registry returned 404 for GET on https://registry.npmjs.org/repo-templates, which is a dependency that verb used to use. This updates verb to the latest version.

ESLint was not running. Updates to 1.4.3.

Changes

  • Updates eslint from 0.21.2 to 1.4.3.
  • Fixes ESLint offenses.
  • Updates verb from ~0.2.0 to ^0.8.10.

Review

Specified verb version was returning error `Registry returned 404 for GET on https://registry.npmjs.org/repo-templates`, which is a dependency that verb used to use. This updates verb to the latest version.
@anselmbradford
Copy link
Member Author

Oops, just noticed this is covered in #32

@anselmbradford
Copy link
Member Author

This PR's still valid though b/c it fixes some warnings and adds missing ESLint rules.

@contolini
Copy link
Member

Dope. I'll pull both these in.

@contolini contolini merged commit c847663 into cfpb:master Sep 17, 2015
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

2 participants