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

Move to using yarn #421

Closed
brettcannon opened this issue Dec 14, 2017 · 0 comments · Fixed by #475
Closed

Move to using yarn #421

brettcannon opened this issue Dec 14, 2017 · 0 comments · Fixed by #475
Assignees
Labels
debt Covers everything internal: CI, testing, refactoring of the codebase, etc.
Milestone

Comments

@brettcannon
Copy link
Member

VS Code has switched, so we might as well follow.

@brettcannon brettcannon added awaiting 2-PR debt Covers everything internal: CI, testing, refactoring of the codebase, etc. labels Dec 14, 2017
@brettcannon brettcannon added this to the January 2018 milestone Dec 14, 2017
@brettcannon brettcannon self-assigned this Dec 14, 2017
DonJayamanne added a commit that referenced this issue Jan 3, 2018
* Fixes #421 (move from npm to yarn)
* Fixes #464 (code coverage of the extension code) 
* Fixes to some unit tests that were failing just on Windows
@lock lock bot locked as resolved and limited conversation to collaborators Jul 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debt Covers everything internal: CI, testing, refactoring of the codebase, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants