You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We decided that linting DESCRIPTION is out-of-scope for {lintr} (#889).
Note
Name {meta.lintr} is placeholder, but it would be nice for the chosen name to reference the fact that this linter package is targeting meta-data about packages; DESCRIPTION is only one type of meta-data, and we could cover other files in future (e.g. CITATION).
Metadata smells
Sort dependencies alphabetically in Imports, Suggests, and Depends
(usethis::use_tidy_dependencies() does this)
Adding Language field (for spell check, either via {spelling} or some other solution)
Recommending lazy loading data if data is shipped in the package (LazyData: true)
Recommending which minimum required R version should be used depending on the R version requirements of hard/runtime dependencies
Omitting hard-coded Date field in DESCRIPTION
Including URL field to specify documentation website or source code repository location
Including BugReports field to specify where bugs can be reported
Out-of-scope
Anything that is already flagged by R CMD check (e.g. setting LazyData: true when there is no data in the package), since it's redundant as most CI systems for R packages are set up to run both R CMD check and linter.
P.S. Feel free to edit and update the issue as you think of more lints that could be covered in this package.
The text was updated successfully, but these errors were encountered:
Preamble
We decided that linting
DESCRIPTION
is out-of-scope for{lintr}
(#889).Note
Name
{meta.lintr}
is placeholder, but it would be nice for the chosen name to reference the fact that this linter package is targeting meta-data about packages;DESCRIPTION
is only one type of meta-data, and we could cover other files in future (e.g.CITATION
).Metadata smells
Imports
,Suggests
, andDepends
(
usethis::use_tidy_dependencies()
does this)Language
field (for spell check, either via{spelling}
or some other solution)LazyData: true
)Date
field inDESCRIPTION
BugReports
field to specify where bugs can be reportedOut-of-scope
Anything that is already flagged by
R CMD check
(e.g. settingLazyData: true
when there is no data in the package), since it's redundant as most CI systems for R packages are set up to run both R CMD check and linter.P.S. Feel free to edit and update the issue as you think of more lints that could be covered in this package.
The text was updated successfully, but these errors were encountered: