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(deps): replace cli-color dep with colorette #117

Merged
merged 7 commits into from Oct 14, 2022
Merged

fix(deps): replace cli-color dep with colorette #117

merged 7 commits into from Oct 14, 2022

Conversation

ghost
Copy link

@ghost ghost commented Oct 14, 2022

What this PR do?

  • Replaces the cli-color dependency with colorette

closes #56

@ghost ghost changed the title improvement: Replace cli-color dep with colorette improvement(deps) Replace cli-color dep with colorette Oct 14, 2022
@ghost ghost changed the title improvement(deps) Replace cli-color dep with colorette fix(deps): Replace cli-color dep with colorette Oct 14, 2022
index.ts Outdated Show resolved Hide resolved
DerekNonGeneric and others added 3 commits October 14, 2022 16:41
index.ts Outdated Show resolved Hide resolved
Copy link
Member

@DerekNonGeneric DerekNonGeneric left a comment

Choose a reason for hiding this comment

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

Looks good to me! I wonder if @jorgebucaran thinks so too. :)

index.ts Outdated Show resolved Hide resolved
@ghost
Copy link
Author

ghost commented Oct 14, 2022

@DerekNonGeneric it seems, that it does not agree with your change about the import. What do you want me to do? leave it as is, and you'll ignore the warning, or rewrite all imports to double quotes so it's consistent and passes the test?

Copy link
Collaborator

@jorgebucaran jorgebucaran left a comment

Choose a reason for hiding this comment

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

Looks great to me!

@DerekNonGeneric
Copy link
Member

leave it as is, and you'll ignore the warning

Yeah, that's what we'll do here; apparently it needs to be reconfigured — not a problem!

@DerekNonGeneric DerekNonGeneric merged commit d4c7db0 into OpenINF:main Oct 14, 2022
@DerekNonGeneric DerekNonGeneric changed the title fix(deps): Replace cli-color dep with colorette fix(deps): replace cli-color dep with colorette Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] dependency cli-color has licensing issues
3 participants