-
Notifications
You must be signed in to change notification settings - Fork 5
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
Format R code to standard #113
Labels
Comments
I like it. From a quick look at the options |
I've done some work with lintr so styler is a better match (they are from
the same dev).
Lintr is very picky so it'll help with general code quality and
consistency.
…On Sun, Jul 9, 2023, 16:48 Santiago Casanova ***@***.***> wrote:
I like it. From a quick look at the options styler (+ lintr) might be the
best choice (?). Do you have a preference?
—
Reply to this email directly, view it on GitHub
<#113 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABJLHRS4UOHYYX4VXLTJLZLXPMKKPANCNFSM6AAAAAA2DEQV3M>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Merged
Sounds good to me. Also, I'm back to the same timezone so that will be nice for collaborating |
I need to validate that styler code is running on PR before closing this. |
In working with #126 I had a commit added by the styler job ( I think) so that's likely working. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Consider formatting R code with
formatR
,styler
and/orlintr
- could do automatically at merge with a Github action.@SCasanova feel free to assign to me if you think it's a good idea, or close if you'd rather not do that.
The text was updated successfully, but these errors were encountered: