Skip to content

Update geom-text.r #4389

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

Merged

Conversation

larspijnappel
Copy link
Contributor

As proposed, here's a PR for the documentation improvement regarding the unsupported check_overlap argument for geom_label() (#4331).

Note: this request replaces the previous PR #4332 as it can't be completed due to loss of the previous fork.

…the unsupported `check_overlap` argument for `geom_label()` (tidyverse#4331).
@thomasp85
Copy link
Member

Can I get you to run roxygen to also update the docs

@larspijnappel
Copy link
Contributor Author

No problem, although that would be another first time for me as well :-)

So, in my local branch I performed devtools::document() and the man\geom_text.Rd has been updated successfully.

Do I just pr_push() again to complete all of it?

@thomasp85
Copy link
Member

Yes - every change can be added to the PR using pr_push()

NAMESPACE Outdated
@@ -489,6 +489,7 @@ export(scale_color_fermenter)
export(scale_color_gradient)
export(scale_color_gradient2)
export(scale_color_gradientn)
export(scale_color_gray)
Copy link
Member

Choose a reason for hiding this comment

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

Can I get you to remove the two new exports in the NAMESPACE... It is not your fault, but another PR did not get oxygenised and this is now causing your PR to fail check. Removing these lets us handle it in a different PR

NAMESPACE Outdated
@@ -536,6 +537,7 @@ export(scale_fill_fermenter)
export(scale_fill_gradient)
export(scale_fill_gradient2)
export(scale_fill_gradientn)
export(scale_fill_gray)
Copy link
Member

Choose a reason for hiding this comment

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

same as above

Copy link
Member

@thomasp85 thomasp85 left a comment

Choose a reason for hiding this comment

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

Thanks!

@thomasp85 thomasp85 merged commit aafcac3 into tidyverse:master Apr 9, 2021
@larspijnappel larspijnappel deleted the geom-text_patch_documentation branch April 12, 2021 15:35
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