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

Replace several stringr-based function calls with base equivalents #2202

Merged
merged 7 commits into from Dec 13, 2022

Conversation

rich-iannone
Copy link
Collaborator

This PR replaces several uses of stringr functions with base equivalents. This is being done in order to eventually remove the dependency on the stringr package.

Copy link
Owner

@yihui yihui left a comment

Choose a reason for hiding this comment

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

Thanks! Now we have four more functions left: str_locate(), str_locate_all(), str_match_all(), and str_extract_all().

@yihui yihui marked this pull request as ready for review December 13, 2022 19:16
@rich-iannone
Copy link
Collaborator Author

Thanks! Also really appreciate that you got rid of str_pad().

@yihui yihui merged commit 1a0f2cc into master Dec 13, 2022
@yihui yihui deleted the stringr-replace branch December 13, 2022 19:27
rich-iannone added a commit that referenced this pull request Dec 22, 2022
* master:
  make negative times 0
  replace highr:::spaces() with strrep() in base R
  support chunk options message/warning = NA
  close #2204: drop the support for cairoDevice
  amend 1a0f2cc: make line_count() a few times faster
  Replace several stringr-based function calls with base equivalents (#2202, #1549)
  see if this fix the ruby gems issue https://github.com/yihui/knitr/actions/runs/3682745993/jobs/6230638404
  stop importing xfun::isFALSE() and define isFALSE() only for R < 3.5.0
  Add labels to the default progress bar and allow users to provide a custom progress bar (#2196)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 14, 2023
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.

None yet

2 participants