Issues • concourse/concourse

Found variable declaration and assignment separately

SCC-S1021

Anti-pattern

Major

4 months ago — 4 years old

Seen in 2 files

3

Use plain channel send or receive

SCC-S1000

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

1

Empty string test can be improved

CRT-A0004

Anti-pattern

Major

4 months ago — 4 years old

Seen in 7 files

8

Redundant statements can be removed

RVV-A0010

Anti-pattern

Major

4 months ago — 3 years old

Seen in 2 files

2

Redundant conversion between `string` and `[]byte`

CRT-A0007

Anti-pattern

Major

4 months ago — 2 years old

Seen in 1 file

1

Redundant else-blocks can be eliminated

RVV-A0009

Anti-pattern

Major

4 months ago — 2 years old

Seen in 5 files

5

Empty slice literal used to declare a variable

GO-W1027

Anti-pattern

Minor

4 months ago — a year old

Seen in 96 files

167

Use `http.NoBody` instead of `nil` in `http.NewRequest` calls

GO-R4001

Anti-pattern

Major

4 months ago — a year old

Seen in 44 files

119

Unnecessary dereference expressions

GO-R4004

Anti-pattern

Major

4 months ago — a year old

Seen in 1 file

7

Use `%q` to quote a string in `fmt.Sprintf` format specifiers

GO-R4002

Anti-pattern

Minor

4 months ago — a year old

Seen in 13 files

27

String concatenation can be simplified

GO-R4003

Anti-pattern

Major

4 months ago — a year old

Seen in 1 file

1

Potential issue in `filepath.Join()` function calls

GO-W4001

Anti-pattern

Major

4 months ago — a year old

Seen in 8 files

17

Replace `for` loop with call to `copy` for slice

SCC-S1001

Anti-pattern

Major

4 months ago — 4 months old

Seen in 1 file

1

Found `time.Now().Sub(t)` instead of `time.Since(t)`

SCC-S1012

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

1

Nested `if` can be replaced with `else if`

CRT-A0011

Anti-pattern

Major

4 months ago — 4 years old

Seen in 4 files

4

Found non-idiomatic returning of boolean expression

SCC-S1008

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

1

Drop unnecessary use of the blank identifier

SCC-S1005

Anti-pattern

Major

4 months ago — 4 years old

Seen in 4 files

4

Simplify `error` construction with `fmt.Errorf`

SCC-S1028

Anti-pattern

Major

4 months ago — 4 years old

Seen in 2 files

3

Omit redundant `nil` check around loop

SCC-S1031

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

1

Simplify slice expression to sliced value itself

CRT-A0016

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

2

Function call can be replaced with helper function

CRT-A0010

Anti-pattern

Major

4 months ago — 4 years old

Seen in 9 files

12

Function literal can be simplified

CRT-A0018

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

1

Simplify regular expression by using raw string literal

SCC-S1007

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

1

Found redundant control flow

SCC-S1023

Anti-pattern

Major

4 months ago — 4 years old

Seen in 1 file

1

`switch` with single case can be rewritten as `if` or `if-else`

CRT-A0014

Anti-pattern

Minor

4 months ago — 4 years old

Seen in 1 file

1

concourse / concourse

Found variable declaration and assignment separately

Use plain channel send or receive

Empty string test can be improved

Redundant statements can be removed

Redundant conversion between string and []byte

Redundant else-blocks can be eliminated

Empty slice literal used to declare a variable

Use http.NoBody instead of nil in http.NewRequest calls

Unnecessary dereference expressions

Use %q to quote a string in fmt.Sprintf format specifiers

String concatenation can be simplified

Potential issue in filepath.Join() function calls

Replace for loop with call to copy for slice

Found time.Now().Sub(t) instead of time.Since(t)

Nested if can be replaced with else if

Found non-idiomatic returning of boolean expression

Drop unnecessary use of the blank identifier

Simplify error construction with fmt.Errorf

Omit redundant nil check around loop

Simplify slice expression to sliced value itself

Function call can be replaced with helper function

Function literal can be simplified

Simplify regular expression by using raw string literal

Found redundant control flow

switch with single case can be rewritten as if or if-else

Redundant conversion between `string` and `[]byte`

Use `http.NoBody` instead of `nil` in `http.NewRequest` calls

Use `%q` to quote a string in `fmt.Sprintf` format specifiers

Potential issue in `filepath.Join()` function calls

Replace `for` loop with call to `copy` for slice

Found `time.Now().Sub(t)` instead of `time.Since(t)`

Nested `if` can be replaced with `else if`

Simplify `error` construction with `fmt.Errorf`

Omit redundant `nil` check around loop

`switch` with single case can be rewritten as `if` or `if-else`