Skip to content

Bump minimum needed R version to 4.0 #1196

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
Closed
IndrajeetPatil opened this issue May 5, 2024 · 2 comments · Fixed by #1197
Closed

Bump minimum needed R version to 4.0 #1196

IndrajeetPatil opened this issue May 5, 2024 · 2 comments · Fixed by #1197
Assignees
@IndrajeetPatil
Labels
Complexity: Low

Comments

@IndrajeetPatil
Copy link
Collaborator

IndrajeetPatil commented May 5, 2024 •
edited
Loading

Since R 4.4 is out now:

Screenshot 2024-05-05 at 17 54 31

Similar to r-lib/lintr#2569
@lorenzwalthert
Copy link
Collaborator

Exactly. There are some things from #581 that we might clean up.

@IndrajeetPatil IndrajeetPatil self-assigned this May 5, 2024
IndrajeetPatil added a commit that referenced this issue May 5, 2024
@IndrajeetPatil
Bump minimum needed R version to 4.0
8b8ed19 
closes #1196
@IndrajeetPatil IndrajeetPatil mentioned this issue May 5, 2024
Merged
@IndrajeetPatil
Copy link
Collaborator Author

The parser version will now always be 3L, so, technically, we could get rid of it.

But, then, it might be good to retain it to be future-proof in case we need to introduce a new version to accommodate some parser changes in future R versions.

@IndrajeetPatil IndrajeetPatil mentioned this issue Jul 3, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@IndrajeetPatil IndrajeetPatil

Labels
Complexity: Low
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bump minimum needed R version to 4.0 r-lib/styler
2 participants
@lorenzwalthert @IndrajeetPatil