mirror of
https://github.com/golangci/golangci-lint-action.git
synced 2025-12-15 23:26:31 +00:00
Compare commits
13 commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
ef750330e2 | ||
|
|
3ab2377cc3 | ||
|
|
ca80beefdd | ||
|
|
87a373f691 | ||
|
|
7fa6da85a5 | ||
|
|
1e7e51e771 | ||
|
|
5256ff0c0a | ||
|
|
13fed6f911 | ||
|
|
7afe8ff863 | ||
|
|
5a9289952f | ||
|
|
aa6fad0ea0 | ||
|
|
a6071aaacb | ||
|
|
6e36c8460f |
32 changed files with 95470 additions and 100167 deletions
9
.github/workflows/codeql.yaml
vendored
9
.github/workflows/codeql.yaml
vendored
|
|
@ -11,6 +11,11 @@ on:
|
||||||
schedule:
|
schedule:
|
||||||
- cron: '0 17 * * 5'
|
- cron: '0 17 * * 5'
|
||||||
|
|
||||||
|
permissions:
|
||||||
|
actions: read
|
||||||
|
contents: read
|
||||||
|
security-events: write
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
codeQL:
|
codeQL:
|
||||||
# CodeQL runs on ubuntu-latest, windows-latest, and macos-latest
|
# CodeQL runs on ubuntu-latest, windows-latest, and macos-latest
|
||||||
|
|
@ -18,7 +23,7 @@ jobs:
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
- name: Checkout repository
|
- name: Checkout repository
|
||||||
uses: actions/checkout@v5
|
uses: actions/checkout@v6
|
||||||
with:
|
with:
|
||||||
# Must fetch at least the immediate parents so that if this is
|
# Must fetch at least the immediate parents so that if this is
|
||||||
# a pull request then we can checkout the head of the pull request.
|
# a pull request then we can checkout the head of the pull request.
|
||||||
|
|
@ -36,7 +41,7 @@ jobs:
|
||||||
uses: github/codeql-action/init@v4
|
uses: github/codeql-action/init@v4
|
||||||
# Override language selection by uncommenting this and choosing your languages
|
# Override language selection by uncommenting this and choosing your languages
|
||||||
with:
|
with:
|
||||||
language: 'javascript'
|
languages: 'javascript-typescript'
|
||||||
|
|
||||||
- run: |
|
- run: |
|
||||||
npm install
|
npm install
|
||||||
|
|
|
||||||
54
.github/workflows/test.yml
vendored
54
.github/workflows/test.yml
vendored
|
|
@ -1,5 +1,6 @@
|
||||||
name: "build-and-test"
|
name: "build-and-test"
|
||||||
on: # rebuild any PRs and main branch changes
|
|
||||||
|
on:
|
||||||
pull_request:
|
pull_request:
|
||||||
branches:
|
branches:
|
||||||
- main
|
- main
|
||||||
|
|
@ -7,22 +8,27 @@ on: # rebuild any PRs and main branch changes
|
||||||
branches:
|
branches:
|
||||||
- main
|
- main
|
||||||
|
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
build: # make sure build/ci work properly
|
# make sure build/ci work properly
|
||||||
|
build:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/setup-node@v6
|
- uses: actions/setup-node@v6
|
||||||
with:
|
with:
|
||||||
node-version: 24.x
|
node-version: 24.x
|
||||||
- uses: actions/checkout@v5
|
- uses: actions/checkout@v6
|
||||||
- run: |
|
- run: |
|
||||||
npm install
|
npm install
|
||||||
npm run all
|
npm run all
|
||||||
|
|
||||||
# Fail the build if there is dirty change
|
# Fail the build if there are changes.
|
||||||
- run: git diff --exit-code -- dist
|
- run: git diff --exit-code -- dist
|
||||||
|
|
||||||
test: # make sure the action works on a clean machine without building
|
# make sure the action works on a clean machine without building
|
||||||
|
test:
|
||||||
needs: [ build ]
|
needs: [ build ]
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
|
|
@ -34,14 +40,14 @@ jobs:
|
||||||
version:
|
version:
|
||||||
- ""
|
- ""
|
||||||
- "latest"
|
- "latest"
|
||||||
- "v2.5"
|
- "v2.7"
|
||||||
- "v2.5.0"
|
- "v2.7.2"
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
permissions:
|
permissions:
|
||||||
contents: read
|
contents: read
|
||||||
pull-requests: read
|
pull-requests: read
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v5
|
- uses: actions/checkout@v6
|
||||||
- uses: actions/setup-node@v6
|
- uses: actions/setup-node@v6
|
||||||
with:
|
with:
|
||||||
node-version: 24.x
|
node-version: 24.x
|
||||||
|
|
@ -51,10 +57,11 @@ jobs:
|
||||||
- uses: ./
|
- uses: ./
|
||||||
with:
|
with:
|
||||||
version: ${{ matrix.version }}
|
version: ${{ matrix.version }}
|
||||||
args: --timeout=5m --issues-exit-code=0 ./sample/...
|
args: --timeout=5m --issues-exit-code=0 ./fixtures/simple/...
|
||||||
only-new-issues: true
|
only-new-issues: true
|
||||||
|
|
||||||
test-go-install: # make sure the action works on a clean machine without building (go-install mode)
|
# make sure the action works on a clean machine without building (go-install mode)
|
||||||
|
test-go-install:
|
||||||
needs: [ build ]
|
needs: [ build ]
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
|
|
@ -66,14 +73,14 @@ jobs:
|
||||||
version:
|
version:
|
||||||
- ""
|
- ""
|
||||||
- "latest"
|
- "latest"
|
||||||
- "v2.5.0"
|
- "v2.7.2"
|
||||||
- "655e8ede5178280b2a640e185bc4a343aed0f54e"
|
- "655e8ede5178280b2a640e185bc4a343aed0f54e"
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
permissions:
|
permissions:
|
||||||
contents: read
|
contents: read
|
||||||
pull-requests: read
|
pull-requests: read
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v5
|
- uses: actions/checkout@v6
|
||||||
- uses: actions/setup-node@v6
|
- uses: actions/setup-node@v6
|
||||||
with:
|
with:
|
||||||
node-version: 24.x
|
node-version: 24.x
|
||||||
|
|
@ -83,7 +90,7 @@ jobs:
|
||||||
- uses: ./
|
- uses: ./
|
||||||
with:
|
with:
|
||||||
version: ${{ matrix.version }}
|
version: ${{ matrix.version }}
|
||||||
args: --timeout=5m --issues-exit-code=0 ./sample/...
|
args: --timeout=5m --issues-exit-code=0 ./fixtures/simple/...
|
||||||
only-new-issues: true
|
only-new-issues: true
|
||||||
install-mode: goinstall
|
install-mode: goinstall
|
||||||
|
|
||||||
|
|
@ -97,13 +104,13 @@ jobs:
|
||||||
- macos-latest
|
- macos-latest
|
||||||
- windows-latest
|
- windows-latest
|
||||||
wd:
|
wd:
|
||||||
- sample-go-mod
|
- fixtures/go-mod
|
||||||
- sample-go-tool
|
- fixtures/go-tool
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
permissions:
|
permissions:
|
||||||
contents: read
|
contents: read
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v5
|
- uses: actions/checkout@v6
|
||||||
- uses: actions/setup-node@v6
|
- uses: actions/setup-node@v6
|
||||||
with:
|
with:
|
||||||
node-version: 24.x
|
node-version: 24.x
|
||||||
|
|
@ -115,7 +122,8 @@ jobs:
|
||||||
working-directory: ${{ matrix.wd }}
|
working-directory: ${{ matrix.wd }}
|
||||||
args: --timeout=5m --issues-exit-code=0 ./...
|
args: --timeout=5m --issues-exit-code=0 ./...
|
||||||
|
|
||||||
test-plugins: # make sure the action works on a clean machine with plugins
|
# make sure the action works on a clean machine with plugins
|
||||||
|
test-plugins:
|
||||||
needs: [ build ]
|
needs: [ build ]
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
|
|
@ -127,14 +135,14 @@ jobs:
|
||||||
version:
|
version:
|
||||||
- ""
|
- ""
|
||||||
- "latest"
|
- "latest"
|
||||||
- "v2.5"
|
- "v2.7"
|
||||||
- "v2.5.0"
|
- "v2.7.2"
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
permissions:
|
permissions:
|
||||||
contents: read
|
contents: read
|
||||||
pull-requests: read
|
pull-requests: read
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v5
|
- uses: actions/checkout@v6
|
||||||
- uses: actions/setup-node@v6
|
- uses: actions/setup-node@v6
|
||||||
with:
|
with:
|
||||||
node-version: 24.x
|
node-version: 24.x
|
||||||
|
|
@ -144,7 +152,7 @@ jobs:
|
||||||
- uses: ./
|
- uses: ./
|
||||||
with:
|
with:
|
||||||
version: ${{ matrix.version }}
|
version: ${{ matrix.version }}
|
||||||
working-directory: sample-plugins
|
working-directory: fixtures/plugins
|
||||||
args: --timeout=5m --issues-exit-code=0 ./...
|
args: --timeout=5m --issues-exit-code=0 ./...
|
||||||
|
|
||||||
test-monorepo:
|
test-monorepo:
|
||||||
|
|
@ -161,7 +169,7 @@ jobs:
|
||||||
contents: read
|
contents: read
|
||||||
pull-requests: read
|
pull-requests: read
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v5
|
- uses: actions/checkout@v6
|
||||||
- uses: actions/setup-node@v6
|
- uses: actions/setup-node@v6
|
||||||
with:
|
with:
|
||||||
node-version: 24.x
|
node-version: 24.x
|
||||||
|
|
@ -170,6 +178,6 @@ jobs:
|
||||||
go-version: oldstable
|
go-version: oldstable
|
||||||
- uses: ./
|
- uses: ./
|
||||||
with:
|
with:
|
||||||
working-directory: sample-monorepo
|
working-directory: fixtures/monorepo
|
||||||
experimental: "automatic-module-directories"
|
experimental: "automatic-module-directories"
|
||||||
args: --timeout=5m --issues-exit-code=0 ./...
|
args: --timeout=5m --issues-exit-code=0 ./...
|
||||||
|
|
|
||||||
53
README.md
53
README.md
|
|
@ -261,21 +261,22 @@ You will also likely need to add the following `.gitattributes` file to ensure t
|
||||||
|
|
||||||
### Overview
|
### Overview
|
||||||
|
|
||||||
| Option | Description |
|
| Option | Description |
|
||||||
|---------------------------------------------------------------|----------------------------------------------------|
|
|---------------------------------------------------------------|-------------------------------------------------------|
|
||||||
| [`version`](#version) | The version of golangci-lint to use. |
|
| [`version`](#version) | The version of golangci-lint to use. |
|
||||||
| [`install-mode`](#install-mode) | The mode to install golangci-lint. |
|
| [`version-file`](#version-file) | Gets the version of golangci-lint to use from a file. |
|
||||||
| [`install-only`](#install-only) | Only install golangci-lint. |
|
| [`install-mode`](#install-mode) | The mode to install golangci-lint. |
|
||||||
| [`verify`](#verify) | Validates golangci-lint configuration file. |
|
| [`install-only`](#install-only) | Only install golangci-lint. |
|
||||||
| [`github-token`](#github-token) | Used by the `only-new-issues` option. |
|
| [`verify`](#verify) | Validates golangci-lint configuration file. |
|
||||||
| [`only-new-issues`](#only-new-issues) | Show only new issues. |
|
| [`github-token`](#github-token) | Used by the `only-new-issues` option. |
|
||||||
| [`working-directory`](#working-directory) | The golangci-lint working directory. |
|
| [`only-new-issues`](#only-new-issues) | Show only new issues. |
|
||||||
| [`args`](#args) | Golangci-lint command line arguments. |
|
| [`working-directory`](#working-directory) | The golangci-lint working directory. |
|
||||||
| [`skip-cache`](#skip-cache) | Disable cache support. |
|
| [`args`](#args) | Golangci-lint command line arguments. |
|
||||||
| [`skip-save-cache`](#skip-save-cache) | Don't save cache. |
|
| [`skip-cache`](#skip-cache) | Disable cache support. |
|
||||||
| [`cache-invalidation-interval`](#cache-invalidation-interval) | Number of days before cache invalidation. |
|
| [`skip-save-cache`](#skip-save-cache) | Don't save cache. |
|
||||||
| [`problem-matchers`](#problem-matchers) | Forces the usage of the embedded problem matchers. |
|
| [`cache-invalidation-interval`](#cache-invalidation-interval) | Number of days before cache invalidation. |
|
||||||
| [Experimental](#experimental) | Experimental options |
|
| [`problem-matchers`](#problem-matchers) | Forces the usage of the embedded problem matchers. |
|
||||||
|
| [Experimental](#experimental) | Experimental options |
|
||||||
|
|
||||||
### Installation
|
### Installation
|
||||||
|
|
||||||
|
|
@ -302,6 +303,28 @@ with:
|
||||||
|
|
||||||
</details>
|
</details>
|
||||||
|
|
||||||
|
#### `version-file`
|
||||||
|
|
||||||
|
Gets the version of golangci-lint to use from a file.
|
||||||
|
|
||||||
|
The path must be relative to the root of the project, or the `working-directory` if defined.
|
||||||
|
|
||||||
|
This parameter supports `.golangci-lint-version`, and `.tool-versions` files.
|
||||||
|
|
||||||
|
Only works with `install-mode: binary` (the default).
|
||||||
|
|
||||||
|
<details>
|
||||||
|
<summary>Example</summary>
|
||||||
|
|
||||||
|
```yml
|
||||||
|
uses: golangci/golangci-lint-action@v9
|
||||||
|
with:
|
||||||
|
version-file: .tool-versions
|
||||||
|
# ...
|
||||||
|
```
|
||||||
|
|
||||||
|
</details>
|
||||||
|
|
||||||
#### `install-mode`
|
#### `install-mode`
|
||||||
|
|
||||||
(optional)
|
(optional)
|
||||||
|
|
|
||||||
|
|
@ -11,6 +11,13 @@ inputs:
|
||||||
- `goinstall`: the value can be v2.3.4, `latest`, or the hash of a commit.
|
- `goinstall`: the value can be v2.3.4, `latest`, or the hash of a commit.
|
||||||
- `none`: the value is ignored.
|
- `none`: the value is ignored.
|
||||||
required: false
|
required: false
|
||||||
|
version-file:
|
||||||
|
description: |
|
||||||
|
Gets the version of golangci-lint to use from a file.
|
||||||
|
The path must be relative to the root of the project, or the `working-directory` if defined.
|
||||||
|
This parameter supports `.golangci-lint-version`, and `.tool-versions` files.
|
||||||
|
Only works with `install-mode: binary` (the default).
|
||||||
|
required: false
|
||||||
install-mode:
|
install-mode:
|
||||||
description: "The mode to install golangci-lint. It can be 'binary', 'goinstall', or 'none'."
|
description: "The mode to install golangci-lint. It can be 'binary', 'goinstall', or 'none'."
|
||||||
default: "binary"
|
default: "binary"
|
||||||
|
|
|
||||||
97217
dist/post_run/index.js
generated
vendored
97217
dist/post_run/index.js
generated
vendored
File diff suppressed because one or more lines are too long
97217
dist/run/index.js
generated
vendored
97217
dist/run/index.js
generated
vendored
File diff suppressed because one or more lines are too long
|
|
@ -1,4 +1,4 @@
|
||||||
module sample
|
module github.com/golangci/sample
|
||||||
|
|
||||||
go 1.24.0
|
go 1.24.0
|
||||||
|
|
||||||
|
|
@ -1,4 +1,4 @@
|
||||||
module sample
|
module github.com/golangci/sample
|
||||||
|
|
||||||
go 1.24.0
|
go 1.24.0
|
||||||
|
|
||||||
1035
package-lock.json
generated
1035
package-lock.json
generated
File diff suppressed because it is too large
Load diff
14
package.json
14
package.json
|
|
@ -27,23 +27,23 @@
|
||||||
"node": ">=24.0.0"
|
"node": ">=24.0.0"
|
||||||
},
|
},
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"@actions/cache": "^4.1.0",
|
"@actions/cache": "^5.0.1",
|
||||||
"@actions/core": "^1.11.1",
|
"@actions/core": "^2.0.1",
|
||||||
"@actions/exec": "^1.1.1",
|
"@actions/exec": "^2.0.0",
|
||||||
"@actions/github": "^6.0.1",
|
"@actions/github": "^6.0.1",
|
||||||
"@actions/http-client": "^3.0.0",
|
"@actions/http-client": "^3.0.0",
|
||||||
"@octokit/plugin-retry": "^6.1.0",
|
"@octokit/plugin-retry": "^6.1.0",
|
||||||
"@actions/tool-cache": "^2.0.2",
|
"@actions/tool-cache": "^2.0.2",
|
||||||
"@types/node": "^24.10.1",
|
"@types/node": "^25.0.2",
|
||||||
"@types/semver": "^7.7.1",
|
"@types/semver": "^7.7.1",
|
||||||
"@types/tmp": "^0.2.6",
|
"@types/tmp": "^0.2.6",
|
||||||
"@types/which": "^3.0.4",
|
"@types/which": "^3.0.4",
|
||||||
"tmp": "^0.2.5",
|
"tmp": "^0.2.5",
|
||||||
"which": "^6.0.0",
|
"which": "^6.0.0",
|
||||||
"yaml": "^2.8.1"
|
"yaml": "^2.8.2"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"@typescript-eslint/eslint-plugin": "^8.46.4",
|
"@typescript-eslint/eslint-plugin": "^8.49.0",
|
||||||
"@typescript-eslint/parser": "^8.32.1",
|
"@typescript-eslint/parser": "^8.32.1",
|
||||||
"@vercel/ncc": "^0.38.4",
|
"@vercel/ncc": "^0.38.4",
|
||||||
"eslint": "^8.57.1",
|
"eslint": "^8.57.1",
|
||||||
|
|
@ -51,7 +51,7 @@
|
||||||
"eslint-plugin-import": "^2.32.0",
|
"eslint-plugin-import": "^2.32.0",
|
||||||
"eslint-plugin-prettier": "^5.5.4",
|
"eslint-plugin-prettier": "^5.5.4",
|
||||||
"eslint-plugin-simple-import-sort": "^12.1.1",
|
"eslint-plugin-simple-import-sort": "^12.1.1",
|
||||||
"prettier": "^3.6.2",
|
"prettier": "^3.7.4",
|
||||||
"typescript": "^5.9.3"
|
"typescript": "^5.9.3"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -67,6 +67,12 @@ const isLessVersion = (a: Version, b: Version): boolean => {
|
||||||
|
|
||||||
const getRequestedVersion = (): Version => {
|
const getRequestedVersion = (): Version => {
|
||||||
let requestedVersion = core.getInput(`version`)
|
let requestedVersion = core.getInput(`version`)
|
||||||
|
let versionFilePath = core.getInput(`version-file`)
|
||||||
|
|
||||||
|
if (requestedVersion && versionFilePath) {
|
||||||
|
core.warning(`Both version (${requestedVersion}) and version-file (${versionFilePath}) inputs are specified, only version will be used`)
|
||||||
|
}
|
||||||
|
|
||||||
const workingDirectory = core.getInput(`working-directory`)
|
const workingDirectory = core.getInput(`working-directory`)
|
||||||
|
|
||||||
let goMod = "go.mod"
|
let goMod = "go.mod"
|
||||||
|
|
@ -83,6 +89,27 @@ const getRequestedVersion = (): Version => {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (requestedVersion == "" && versionFilePath) {
|
||||||
|
if (workingDirectory) {
|
||||||
|
versionFilePath = path.join(workingDirectory, versionFilePath)
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!fs.existsSync(versionFilePath)) {
|
||||||
|
throw new Error(`The specified golangci-lint version file at: ${versionFilePath} does not exist`)
|
||||||
|
}
|
||||||
|
|
||||||
|
const content = fs.readFileSync(versionFilePath, "utf-8")
|
||||||
|
|
||||||
|
if (path.basename(versionFilePath) === ".tool-versions") {
|
||||||
|
// asdf/mise file.
|
||||||
|
const match = content.match(/^golangci-lint\s+([^\n#]+)/m)
|
||||||
|
requestedVersion = match ? "v" + match[1].trim().replace(/^v/gi, "") : ""
|
||||||
|
} else {
|
||||||
|
// .golangci-lint-version file.
|
||||||
|
requestedVersion = "v" + content.trim().replace(/^v/gi, "")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
const parsedRequestedVersion = parseVersion(requestedVersion)
|
const parsedRequestedVersion = parseVersion(requestedVersion)
|
||||||
if (parsedRequestedVersion == null) {
|
if (parsedRequestedVersion == null) {
|
||||||
return null
|
return null
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue