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.

Sign up for GitHub

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

Jump to bottom

Replace crayon functions by cli #233

Closed
krlmlr opened this issue Jan 25, 2021 · 2 comments · Fixed by #406
Closed

Replace crayon functions by cli #233

krlmlr opened this issue Jan 25, 2021 · 2 comments · Fixed by #406
Milestone
1.6.5

Comments

@krlmlr
Copy link
Member

krlmlr commented Jan 25, 2021

when cli::num_ansi_colors() becomes available.

CC @gaborcsardi.
@krlmlr krlmlr added this to the 1.6.5 milestone Oct 25, 2021
@krlmlr
Copy link
Member Author

krlmlr commented Dec 25, 2021

Requires cli >= 2.3.0, already on CRAN.

@krlmlr krlmlr mentioned this issue Dec 26, 2021
Merged
krlmlr added a commit that referenced this issue Dec 26, 2021
@krlmlr
Merge pull request #406 from r-lib/f-233-cli
bc5ba18 
- Remove dependency on the crayon package (#233, #406).
@github-actions
Copy link
Contributor

This old thread has been automatically locked. If you think you have found something related to this, please open a new issue and link to this old issue if necessary.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.6.5
Development

Successfully merging a pull request may close this issue.

Remove crayon dependency r-lib/pillar
1 participant
@krlmlr