mirror of
https://github.com/rclone/rclone.git
synced 2024-11-07 09:04:52 +01:00
build: ignore remaining gocritic lint issues
This commit is contained in:
parent
4d787ae87f
commit
330c6702eb
@ -42,11 +42,11 @@ func TestTimestampIsZero(t *testing.T) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func TestTimestampEqual(t *testing.T) {
|
func TestTimestampEqual(t *testing.T) {
|
||||||
assert.False(t, emptyT.Equal(emptyT))
|
assert.False(t, emptyT.Equal(emptyT)) //nolint:gocritic // Don't include gocritic when running golangci-lint to avoid dupArg: suspicious method call with the same argument and receiver
|
||||||
assert.False(t, t0.Equal(emptyT))
|
assert.False(t, t0.Equal(emptyT))
|
||||||
assert.False(t, emptyT.Equal(t0))
|
assert.False(t, emptyT.Equal(t0))
|
||||||
assert.False(t, t0.Equal(t1))
|
assert.False(t, t0.Equal(t1))
|
||||||
assert.False(t, t1.Equal(t0))
|
assert.False(t, t1.Equal(t0))
|
||||||
assert.True(t, t0.Equal(t0))
|
assert.True(t, t0.Equal(t0)) //nolint:gocritic // Don't include gocritic when running golangci-lint to avoid dupArg: suspicious method call with the same argument and receiver
|
||||||
assert.True(t, t1.Equal(t1))
|
assert.True(t, t1.Equal(t1)) //nolint:gocritic // Don't include gocritic when running golangci-lint to avoid dupArg: suspicious method call with the same argument and receiver
|
||||||
}
|
}
|
||||||
|
@ -3392,7 +3392,7 @@ func setQuirks(opt *Options) {
|
|||||||
// See: https://issuetracker.google.com/issues/323465186
|
// See: https://issuetracker.google.com/issues/323465186
|
||||||
// So make cutoff very large which it does seem to support
|
// So make cutoff very large which it does seem to support
|
||||||
opt.CopyCutoff = math.MaxInt64
|
opt.CopyCutoff = math.MaxInt64
|
||||||
default:
|
default: //nolint:gocritic // Don't include gocritic when running golangci-lint to avoid defaultCaseOrder: consider to make `default` case as first or as last case
|
||||||
fs.Logf("s3", "s3 provider %q not known - please set correctly", opt.Provider)
|
fs.Logf("s3", "s3 provider %q not known - please set correctly", opt.Provider)
|
||||||
fallthrough
|
fallthrough
|
||||||
case "Other":
|
case "Other":
|
||||||
|
Loading…
Reference in New Issue
Block a user