Skip to content

Commit

Permalink
Include more linters in .lintr_new (#1593)
Browse files Browse the repository at this point in the history
Also

- Sync up with `lint.yaml` changes
- List linters alphabetically
  • Loading branch information
IndrajeetPatil authored Oct 3, 2022
1 parent 481a158 commit cc79867
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 6 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/lint-changed-files.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
on:
pull_request:
branches:
- main
- master
branches: [main, master]

name: lint-changed-files

Expand All @@ -19,6 +17,7 @@ jobs:
- uses: r-lib/actions/setup-r-dependencies@v2
with:
extra-packages: |
r-lib/lintr
any::gh
any::purrr
needs: check
Expand Down
14 changes: 11 additions & 3 deletions .lintr_new
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
linters: linters_with_defaults(
implicit_integer_linter(),
line_length_linter(120),
any_duplicated_linter(),
any_is_na_linter(),
backport_linter("oldrel-4", except = "R_user_dir"),
consecutive_stopifnot_linter(),
expect_comparison_linter(),
expect_length_linter(),
expect_named_linter(),
Expand All @@ -10,7 +12,13 @@ linters: linters_with_defaults(
expect_s4_class_linter(),
expect_true_false_linter(),
expect_type_linter(),
backport_linter("oldrel-4", except = "R_user_dir")
implicit_integer_linter(),
line_length_linter(120),
missing_argument_linter(),
nested_ifelse_linter(),
numeric_leading_zero_linter(),
redundant_equals_linter(),
redundant_ifelse_linter()
)
exclusions: list(
"inst/doc/creating_linters.R" = 1,
Expand Down

0 comments on commit cc79867

Please sign in to comment.