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

lintr flags #41

Closed
wibeasley opened this issue Jan 3, 2020 · 0 comments
Closed

lintr flags #41

wibeasley opened this issue Jan 3, 2020 · 0 comments
Assignees
@wibeasley

Comments

@wibeasley
Copy link
Contributor

Offshoot of #38: address some of the lintr flags
@wibeasley wibeasley self-assigned this Jan 3, 2020
@wibeasley wibeasley mentioned this issue Jan 3, 2020
Closed
11 tasks
wibeasley added a commit that referenced this issue Feb 24, 2020
@wibeasley
2-space indent; parameters on separate lines
5ec4d7f 
ref #41
wibeasley added a commit that referenced this issue Jan 1, 2021
@wibeasley
seq_len
c6db664 
--address some lintr flags; ref #41
wibeasley added a commit that referenced this issue Jan 1, 2021
@wibeasley
remove unused magrittr imports
453f231 
--address some lintr flags; ref #41
wibeasley added a commit that referenced this issue Jan 2, 2021
@wibeasley
avoid @importFrom
7e09a81 
From https://r-pkgs.org/namespace.html#import-r

> If you are using just a few functions from another package, my recommendation is to note the package name in the Imports: field of the DESCRIPTION file and call the function(s) explicitly using ::, e.g., pkg::fun().

ref #41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wibeasley wibeasley

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@wibeasley