Skip to content

Commit c4f6899

Browse files
ldezldez
authored
chore: remove analyzer (#222)
* fix: add missing flags Signed-off-by: Fernandez Ludovic <[email protected]> * fix: prefix section parsing Signed-off-by: Fernandez Ludovic <[email protected]> * fix: add an human understanable report message Signed-off-by: Fernandez Ludovic <[email protected]> * fix: golangci-lint configuration Signed-off-by: Fernandez Ludovic <[email protected]> * chore: remove analyzer Signed-off-by: Fernandez Ludovic <[email protected]> * chore: clean workflow Signed-off-by: Fernandez Ludovic <[email protected]> * chore: split workflow Signed-off-by: Fernandez Ludovic <[email protected]> --------- Signed-off-by: Fernandez Ludovic <[email protected]>
1 parent e67c7db commit c4f6899

File tree

10 files changed

+59
-498
lines changed

10 files changed

+59
-498
lines changed

‎.github/workflows/build.yml‎

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
name: build
2+
3+
on:
4+
pull_request:
5+
6+
jobs:
7+
build:
8+
name: Build and Tests
9+
runs-on: ${{ matrix.os }}
10+
env:
11+
GOPROXY: https://proxy.golang.org
12+
ARCHIVE_OUTDIR: dist/archives
13+
TEST_OUTPUT_FILE_PREFIX: test_report
14+
strategy:
15+
matrix:
16+
os: [ubuntu-latest, windows-latest, macos-latest]
17+
go: [ oldstable, stable ]
18+
steps:
19+
- name: Check out code into the Go module directory
20+
uses: actions/checkout@v4
21+
- name: Set up Go
22+
uses: actions/setup-go@v5
23+
with:
24+
go-version: ${{ matrix.go }}
25+
- name: Run make test
26+
env:
27+
COVERAGE_OPTS: "-coverprofile=coverage.txt -covermode=atomic"
28+
run: make test
29+
- name: Run make build
30+
run: make build
31+

‎.github/workflows/gci.yml‎

Lines changed: 0 additions & 82 deletions
This file was deleted.

‎.github/workflows/lint.yml‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
name: lint
2+
3+
on:
4+
pull_request:
5+
6+
jobs:
7+
lint:
8+
runs-on: ubuntu-latest
9+
env:
10+
GOVER: oldstable
11+
GOPROXY: https://proxy.golang.org
12+
steps:
13+
- name: Check out code into the Go module directory
14+
uses: actions/checkout@v4
15+
- name: Set up Go
16+
uses: actions/setup-go@v5
17+
with:
18+
go-version: ${{ env.GOVER }}
19+
- name: golangci-lint
20+
uses: golangci/golangci-lint-action@v6
21+
with:
22+
version: ${{ env.GOLANGCILINT_VER }}
23+

‎.golangci.yml‎

Lines changed: 4 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -4,34 +4,20 @@ run:
44
concurrency: 4
55

66
# timeout for analysis, e.g. 30s, 5m, default is 1m
7-
deadline: 10m
7+
timeout: 10m
88

99
# exit code when at least one issue was found, default is 1
1010
issues-exit-code: 1
1111

1212
# include test files or not, default is true
1313
tests: true
1414

15-
# list of build tags, all linters use it. Default is empty list.
16-
build-tags:
17-
18-
# which dirs to skip: they won't be analyzed;
19-
# can use regexp here: generated.*, regexp is applied on full path;
20-
# default value is empty list, but next dirs are always skipped independently
21-
# from this option's value:
22-
# third_party$, testdata$, examples$, Godeps$, builtin$
23-
skip-dirs:
24-
25-
# which files to skip: they will be analyzed, but issues from them
26-
# won't be reported. Default value is empty list, but there is
27-
# no need to include all autogenerated files, we confidently recognize
28-
# autogenerated files. If it's not please let us know.
29-
skip-files:
3015

3116
# output configuration options
3217
output:
3318
# colored-line-number|line-number|json|tab|checkstyle, default is "colored-line-number"
34-
format: tab
19+
formats:
20+
- format: tab
3521

3622
# print lines of code with issue, default is true
3723
print-issued-lines: true
@@ -59,11 +45,6 @@ linters-settings:
5945
- default # Contains all imports that could not be matched to another section type.
6046
- prefix(github.com/daixiang0/gci) # Groups all imports with the specified Prefix.
6147

62-
# Separators that should be present between sections.
63-
# Default: ["newLine"]
64-
section-separators:
65-
- newLine
66-
6748
gofmt:
6849
# simplify code: gofmt with `-s` option, true by default
6950
simplify: true
@@ -73,13 +54,9 @@ linters-settings:
7354
local-prefixes: github.com/daixiang0/gci
7455

7556
linters:
76-
fast: false
57+
disable-all: true
7758
enable:
7859
- gofmt
7960
- gofumpt
8061
- goimports
8162
- gci
82-
disable-all: true
83-
84-
issues:
85-
exclude:

‎pkg/analyzer/analyzer.go‎

Lines changed: 0 additions & 149 deletions
This file was deleted.

0 commit comments

Comments
 (0)