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

Feature/move tests to jest #134

Closed
wants to merge 5 commits into from

Conversation

PixnBits
Copy link
Contributor

commits from #131 and #133 but switches the tests from karma to jest's jsdom to avoid the need for saucelabs
also runs linting after the unit tests, and adds a "build" script
potential watchout point: updates React from 14 to 16
🤞 for a good Travis build

correct the README, bump for travis builds as well but only LTS (skip 9)
node built-in modules not needed for d3-request dependency
3.13.1 does not have pre-compiled binaries for node 8.x or 10.x
per https://github.com/sass/node-sass/releases/tag/v4.0.0 4.0.0 is backwards compatible with 3.13.1
@PixnBits
Copy link
Contributor Author

PixnBits commented Dec 10, 2018

I noticed node-sass is a bit out of date, I'll update it to save Travis time (via compilation of C).
[edit] went from 6 minutes to 2 👍

@kehess99
Copy link
Contributor

Closing this request. The changes were merged into the updated react update tests PR in order to combine them with our newest changes.

@kehess99 kehess99 closed this Nov 15, 2020
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