Rstudio progress bar handlers don't work by default
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 55/100
Research direction
Start at cli:::builtin_handler_rstudio$able() and inspect how it compares .Platform$GUI with the expected RStudio value. The work is done when options(cli.progress_handlers='rstudio') recognizes the RStudio environment instead of requiring progress_handlers_only.
Written by the indexing model from the issue text.
Description
Using options(cli.progress_handlers='rstudio') doesn't work, due to cli:::builtin_handler_rstudio$able() returning FALSE.
Looking at the code for the function, it is checking that .Platform$GUI == "Rstudio", while for me at least (runnning RStudio 2022.07), it is "RStudio", with a capital S.
At the moment I can work around by doing options(cli.progress_handlers_only='rstudio'), as that bypasses the $able checks, but it would be better to just do a case-insensitive check (assuming that it was indeed Rstudio at some point in the past).
Thanks,
Charles
- Dominant language
- R
- Stars
- 726
- Forks
- 94
- Avg merge
- 3h 35m
- Merged PRs (30d)
- 1
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from r-lib/cli
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 1/5 Under an hour Newbie friendliness 68/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
-
feature
Difficulty 3/5 1-2 days Newbie friendliness 68/100
Similar issues
-
documentation pkg infrastructure
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
epiverse-trace/epiparameter#511 ·
-
function:write_dwc
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Urgent request: Due to vulnerabilities move to API version 12.6.1 (12.6.2 eventually) or 13.1.1 Open
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
jbkunst/highcharter#849 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
r-lib/pkgdepends#485 · 3 comments ·
-
Difficulty 1/5 Under an hour Newbie friendliness 92/100