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 dependencies #978

Merged
merged 4 commits into from
Feb 16, 2019
Merged

Fix dependencies #978

merged 4 commits into from
Feb 16, 2019

Conversation

GeeWee
Copy link
Collaborator

@GeeWee GeeWee commented Feb 16, 2019

It seems like we've managed to break master in some interesting ways.
Due to symlink issues on Windows, it was actually impossible to get the tests running locally, and we had a tslint rule failing on master, which is why a lot of the Dependabot PR's were failing.

I've fixed the windows symlink issues, and updated all major dependencies, except for jest 24, which I'll leave to #957

@GeeWee GeeWee mentioned this pull request Feb 16, 2019
@coveralls
Copy link

Pull Request Test Coverage Report for Build 2379

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.943%

Totals Coverage Status
Change from base Build 2329: 0.0%
Covered Lines: 1111
Relevant Lines: 1163

💛 - Coveralls

@GeeWee
Copy link
Collaborator Author

GeeWee commented Feb 16, 2019

@huafu @kulshekhar I'd love if one of you could merge this in.

@kulshekhar
Copy link
Owner

Thanks for fixing this!

I'm wondering whether this is what caused tests on AppVeyor to fail for the Jest 24 PR.

@kulshekhar kulshekhar merged commit fb53685 into master Feb 16, 2019
@kulshekhar kulshekhar deleted the fix-updates branch February 16, 2019 11:18
@GeeWee
Copy link
Collaborator Author

GeeWee commented Feb 16, 2019

It might have been, it's definitely what's been causing all the dependabot tests to fail :)

@ahnpnl
Copy link
Collaborator

ahnpnl commented Feb 21, 2019

observe today that Travis failed due to ignore package. Strange that CircleCI ran fine

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

4 participants