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

Bump dependencies, drop support for node < 4 #216

Merged
merged 8 commits into from
Jul 21, 2017

Conversation

sudo-suhas
Copy link
Contributor

@sudo-suhas sudo-suhas commented Jul 20, 2017

Changelog

  • Bump dependencies, details below
  • Switch to fs-extra from fs-promise(deprecated)
  • Drop support for node < 4. yargs does not support node < 4
  • Remove unused dependency node-version-check
  • Remove node 0.12 and add node 8 for travis, appveyor builds
  • Update npm script for test-coverage(node_modules/cross-env/bin/cross-env.jsnode_modules/cross-env/dist/bin/cross-env.js)

Updated dependencies

Dependencies

Package Current Latest Dev Dep? Reason
babel-plugin-istanbul 3.1.2 4.1.4 Yes drop Node 0.10 and 0.12 support, add includeUntested option, upgrade dependencies🔗
cross-env 3.2.4 5.0.1 Yes : needs to be escaped on windows, %var% will not longer be converted to $var on unix, : or ; in env var value will be converted to : on UNIX systems or ; on Windows systems, Scripts using quotes or escape sequences will see a difference in behavior🔗
eslint-config-airbnb 13.0.0 15.0.2 Yes enable react/no-array-index-key, react/require-default-props, set default React version to 0.15, update deps, enable new rules🔗
eslint-plugin-react 6.10.3 7.1.0 Yes Update rules for React 15.5.0, Drop Node.js < 4 support, add rules to recommended, remove deprecated, some new rules🔗
node-sass 3.13.1 4.5.3 Yes LibSass 3.5.0.beta.2, Alpine Linux support, CITGM support, Fix installation error on Node 8🔗
nyc 10.0.0 11.0.3 Yes Drop supprt for node < 4, allow .nycrc.json, support for presets, support for per file coverage checking🔗
typescript 1.8.10 2.4.1 Yes Breaking changes
yargs 6.6.0 8.0.2 No extends key in config file is now used for extending other config files, environment variables now take precedence over config files, context now takes precedence over argv and defaults, the arguments passed to functions are now validated, yargs will no longer aggressively suppress errors, Node 4+ is now required🔗

Not updated

Package Current Latest Reason Why not updated
eslint 3.19.0 4.2.0 New rules in recommended, more strict🔗 Peer dependency for eslint-config-airbnb
eslint-plugin-jsx-a11y 2.2.3 6.0.2 Update all rule schemas to accept objects, Add new rules, Removed support for ESLint ^2.10.2🔗 Peer depencency for eslint-config-airbnb

eslint-plugin-jsx-a11y was updated to 5.1.1 as eslint-config-airbnb has the peer dependency "eslint-plugin-jsx-a11y": "^5.1.1".
eslint-plugin-import was updated as well(2.2.02.6.1) again for eslint-config-airbnb

The fs-promise module is now deprecated and this switches to fs-extra

Closes #215

@codecov
Copy link

codecov bot commented Jul 20, 2017

Codecov Report

Merging #216 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #216   +/-   ##
=======================================
  Coverage   98.14%   98.14%           
=======================================
  Files          24       24           
  Lines         432      432           
=======================================
  Hits          424      424           
  Misses          8        8

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da3ba1f...92f5c39. Read the comment docs.

Copy link
Member

@lijunle lijunle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks awesome! Super thank you!

.travis.yml Outdated
@@ -3,10 +3,10 @@ sudo: false
language: node_js

node_js:
- 0.12
- 4
- 6
- 7
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could drop version 7 now.

@sudo-suhas
Copy link
Contributor Author

@lijunle 😄 Glad I could help

@lijunle lijunle merged commit 1a675b4 into depcheck:master Jul 21, 2017
@sudo-suhas sudo-suhas deleted the chore_bump_deps branch July 21, 2017 14:27
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