Skip to content

Commit 8101a34

Browse files
authored
fix(ci): add lint and lint-sync workflows and update golangci.yml (#651)
1 parent 6f3d765 commit 8101a34

File tree

5 files changed

+16
-98
lines changed

5 files changed

+16
-98
lines changed

.github/workflows/lint-soft.yml

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

.github/workflows/lint-sync.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
name: lint-sync
2+
on:
3+
schedule:
4+
# every Sunday at midnight
5+
- cron: "0 0 * * 0"
6+
workflow_dispatch: # allows manual triggering
7+
8+
permissions:
9+
contents: write
10+
pull-requests: write
11+
12+
jobs:
13+
lint:
14+
uses: charmbracelet/meta/.github/workflows/lint-sync.yml@main

.github/workflows/lint.yml

Lines changed: 2 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -3,26 +3,6 @@ on:
33
push:
44
pull_request:
55

6-
permissions:
7-
contents: read
8-
# Optional: allow read access to pull request. Use with `only-new-issues` option.
9-
pull-requests: read
10-
116
jobs:
12-
golangci:
13-
name: lint
14-
runs-on: ubuntu-latest
15-
steps:
16-
- name: Install Go
17-
uses: actions/setup-go@v5
18-
with:
19-
go-version: ^1.18
20-
21-
- uses: actions/checkout@v4
22-
- name: golangci-lint
23-
uses: golangci/golangci-lint-action@v6
24-
with:
25-
# Optional: golangci-lint command line arguments.
26-
#args:
27-
# Optional: show only new issues if it's a pull request. The default value is `false`.
28-
only-new-issues: true
7+
lint:
8+
uses: charmbracelet/meta/.github/workflows/lint.yml@main

.golangci-soft.yml

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

.golangci.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ issues:
1515
linters:
1616
enable:
1717
- bodyclose
18-
- copyloopvar
1918
- gofumpt
2019
- goimports
2120
- gosec
@@ -27,6 +26,3 @@ linters:
2726
- unconvert
2827
- unparam
2928
- whitespace
30-
# We're disabling predeclared because the linter is running a newer version
31-
# of Go than then Go module.
32-
# - predeclared

0 commit comments

Comments
 (0)