Skip to content

Export datetime_scale()#5174

Merged
teunbrand merged 2 commits intotidyverse:mainfrom
teunbrand:datetime_scale
Feb 9, 2023
Merged

Export datetime_scale()#5174
teunbrand merged 2 commits intotidyverse:mainfrom
teunbrand:datetime_scale

Conversation

@teunbrand
Copy link
Copy Markdown
Collaborator

This PR aims to fix #4701.

Briefly, it exports the formerly internal datetime_scale() for use in extension packages. Parameter documentation is largely inherited from other functions, but the trans argument suggests date/time transforms.

Copy link
Copy Markdown
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.

LGTM

@teunbrand teunbrand merged commit 16026c5 into tidyverse:main Feb 9, 2023
@teunbrand teunbrand deleted the datetime_scale branch February 9, 2023 14:56
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.

Export datetime_scale

3 participants