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

Fix Travis CI builds as well as update dependencies to latest versions #199

Closed
wants to merge 3 commits into from
Closed

Conversation

rawr51919
Copy link

This PR may be a big one, but all build tests should now pass just fine.

@rawr51919 rawr51919 changed the title Fix everything with the files so the tests will pass Fix Travis CI builds as well as update dependencies to latest versions Jan 24, 2021
@rawr51919
Copy link
Author

This PR also requires Node 10+. Builds ran using Node 10/11 should pass OK, as long as saucelabs is configured correctly here

index.js Show resolved Hide resolved
@ljharb
Copy link
Member

ljharb commented Jan 24, 2021

Requiring node 10+ is a breaking change that is best avoided, especially for a bundler polyfill like this one.

@rawr51919
Copy link
Author

Superseding this PR with a new one.

@rawr51919
Copy link
Author

This PR has been superseded by PR #200.

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