Skip to content
This repository was archived by the owner on Dec 10, 2021. It is now read-only.

Convert project from Yarn to npm#5

Merged
wrumsby merged 13 commits intomasterfrom
wrumsby/npm
May 7, 2019
Merged

Convert project from Yarn to npm#5
wrumsby merged 13 commits intomasterfrom
wrumsby/npm

Conversation

@wrumsby
Copy link
Copy Markdown

@wrumsby wrumsby commented May 6, 2019

Using synp converted project from Yarn to npm since npm is our standard tooling.

@wrumsby wrumsby requested review from Deadarius and lwcooper May 6, 2019 20:53
@lwcooper
Copy link
Copy Markdown

lwcooper commented May 6, 2019

did you want an .nvmrc file too?

@@ -1,4 +1,4 @@
import * as c from 'chalk';
import chalk from 'chalk';
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like more than just updating from yarn to npm?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This broke linting... somehow

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah ok, fair enough. weird!

Comment thread test/project/openapi.yml
@wrumsby wrumsby merged commit 4b1bf36 into master May 7, 2019
@wrumsby wrumsby deleted the wrumsby/npm branch May 7, 2019 02:16
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.

2 participants