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

Rename use_tidy_style() to use_styler() #2110

Open
DavisVaughan opened this issue Mar 4, 2025 · 2 comments
Open

Rename use_tidy_style() to use_styler() #2110

DavisVaughan opened this issue Mar 4, 2025 · 2 comments
Labels
feature a feature request or enhancement tidy ✨ tools for the tidyverse team

Comments

@DavisVaughan
Copy link
Member

Now that we have use_air() as well #2109

Will probably require a little reorganizing of the file that use_tidy_style() lives in, and the pkgdown section it lives in
@jennybc
Copy link
Member

jennybc commented Mar 4, 2025

Is this what we're thinking?

use_tidy_style() becomes basically a shell that does nothing but messages about current alternatives: use_air() and (a new function) use_styler(). use_styler() gets the existing use_tidy_style() logic.

@jennybc jennybc added feature a feature request or enhancement tidy ✨ tools for the tidyverse team labels Mar 4, 2025
@DavisVaughan
Copy link
Member Author

DavisVaughan commented Mar 4, 2025 •
edited
Loading

Or use_tidy_style() is removed entirely, I'm not sure, but what you suggest also sounds reasonable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature a feature request or enhancement tidy ✨ tools for the tidyverse team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jennybc @DavisVaughan