Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

Fix CI #71

Merged
merged 3 commits into from
Jul 24, 2017
Merged

Fix CI #71

merged 3 commits into from
Jul 24, 2017

Conversation

kriskowal
Copy link
Contributor

This change incorporates necessary fixes to get CI to pass after this project’s long run without changes.

  • npm no longer supports Node 0.10, so we remove this version from CI.
  • The repository location has changed. There are tests that depend on the the git origin.
  • Addresses a breaking change in JSON reader library. The underlying library changed an error message, so this weakens the assertion on the error message.

@kriskowal
Copy link
Contributor Author

These changes have been reviewed in the context of #70. I will merge this without review once tests pass.

@kriskowal kriskowal merged commit d3d878d into dev Jul 24, 2017
@kriskowal kriskowal deleted the fix-ci branch July 24, 2017 17:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant