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

Suggestion: use native pipe, drop magrittr dependency #213

Open
4 tasks
florianm opened this issue May 29, 2024 · 0 comments
Open
4 tasks

Suggestion: use native pipe, drop magrittr dependency #213

florianm opened this issue May 29, 2024 · 0 comments

Comments

@florianm
Copy link
Contributor

ckanr has very few dependencies, which is great for maintainability.

This is a suggestion to remove the dependency on magrittr by using the native pipe.
This would require:

  • Update DESCRIPTION to Require R >=4.1.0
  • Update any code using magrittr::%>% to use the native pipe |>
  • Update DESCRIPTION to remove magrittr from Imports
  • Run tests to verify we didn't break anything

References

@florianm florianm mentioned this issue May 29, 2024
11 tasks
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

No branches or pull requests

1 participant