build: re-enable revive linters

In this commit:

75dfdbf211 ci: revert revive settings back to fix lint

We accidentally disabled all the revive linters. Unfortunately setting
the rules clears the default set of rules so it is necessary to
mention all rules that we need.
This commit is contained in:
Nick Craig-Wood 2023-08-29 12:58:05 +01:00
parent 967fc6d7f4
commit feaa20d885

View File

@ -33,23 +33,67 @@ issues:
- staticcheck - staticcheck
text: 'SA1019: "github.com/rclone/rclone/cmd/serve/httplib" is deprecated' text: 'SA1019: "github.com/rclone/rclone/cmd/serve/httplib" is deprecated'
# don't disable the revive messages about comments on exported functions
include:
- EXC0012
- EXC0013
- EXC0014
- EXC0015
run: run:
# timeout for analysis, e.g. 30s, 5m, default is 1m # timeout for analysis, e.g. 30s, 5m, default is 1m
timeout: 10m timeout: 10m
linters-settings: linters-settings:
revive: revive:
# setting rules seems to disable all the rules, so re-enable them here
rules: rules:
- name: unreachable-code - name: blank-imports
disabled: true disabled: false
- name: unused-parameter - name: context-as-argument
disabled: true disabled: false
- name: context-keys-type
disabled: false
- name: dot-imports
disabled: false
- name: empty-block - name: empty-block
disabled: true disabled: true
- name: error-naming
disabled: false
- name: error-return
disabled: false
- name: error-strings
disabled: false
- name: errorf
disabled: false
- name: exported
disabled: false
- name: increment-decrement
disabled: true
- name: indent-error-flow
disabled: false
- name: package-comments
disabled: false
- name: range
disabled: false
- name: receiver-naming
disabled: false
- name: redefines-builtin-id - name: redefines-builtin-id
disabled: true disabled: true
- name: superfluous-else - name: superfluous-else
disabled: true disabled: true
- name: time-naming
disabled: false
- name: unexported-return
disabled: false
- name: unreachable-code
disabled: true
- name: unused-parameter
disabled: true
- name: var-declaration
disabled: false
- name: var-naming
disabled: false
stylecheck: stylecheck:
# Only enable the checks performed by the staticcheck stand-alone tool, # Only enable the checks performed by the staticcheck stand-alone tool,
# as documented here: https://staticcheck.io/docs/configuration/options/#checks # as documented here: https://staticcheck.io/docs/configuration/options/#checks