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

Pin chalk dependency to 2.1.0 #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

logan
Copy link
Contributor

@logan logan commented Oct 16, 2018

Since chalk 2.2.0, chalk declares types in package.json, which
overrides the @types/chalk dependency and requires a default import.

See chalk/chalk#215 for more context.

@samccone
Copy link
Owner

Unfortunately due to the current runtime env of this tool within google, I need to be locked on 2.1 for now.

Since chalk 2.2.0, chalk declares types in package.json, which
overrides the @types/chalk dependency and requires a default import.
Since this technically breaks backwards compatibility, replace the
caret range with a specific version number.
@logan logan changed the title Use default import for chalk Pin chalk dependency to 2.1.0 Oct 22, 2018
@logan
Copy link
Contributor Author

logan commented Oct 22, 2018

Google, what a silly place.

Here's an alternate approach to resolving the dev-side dilemma.

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