Compare commits
432 Commits
Author | SHA1 | Date | |
---|---|---|---|
5f1fec7010 | |||
601007b788 | |||
d2a913e97b | |||
7233bd71cb | |||
687f029324 | |||
f9990cd216 | |||
f30aa514f9 | |||
6b21f586ed | |||
535ed3a04b | |||
0078ef00ab | |||
776a285c4c | |||
0d51521d42 | |||
65ea3995bb | |||
653016502e | |||
4cb9697578 | |||
dd04c182af | |||
8c36daf744 | |||
0f97d97f47 | |||
27e48c2073 | |||
65466a27fa | |||
520b21135e | |||
369d614672 | |||
240da36b3d | |||
6897b1a051 | |||
4042cfe2d4 | |||
97bb55a183 | |||
744562230b | |||
1e0fb9b015 | |||
8bb781f27a | |||
c60079adbf | |||
f6781576ae | |||
c7b6fc977f | |||
579a4e6741 | |||
32a046958e | |||
7641ddf8a8 | |||
7261544346 | |||
b26535e6d1 | |||
e1c0a0543f | |||
2fdccc41f5 | |||
db0ea3f163 | |||
ed929be3ec | |||
76eaf57237 | |||
0fd20142bb | |||
3fc120753c | |||
8c99775d4b | |||
ea65405b36 | |||
7122616b22 | |||
39b10dd0d9 | |||
4593d99d2e | |||
30dacc8af6 | |||
464d9fd432 | |||
92ba55cf0d | |||
7cd2f0393b | |||
5077eeb6c1 | |||
d6d0cce229 | |||
e92337f3b6 | |||
bf4120ff3c | |||
61703f1af4 | |||
8b76d976da | |||
a2dc749409 | |||
aa94bc705d | |||
fdf641b40e | |||
12dfd24d9a | |||
01dfc8644b | |||
1ea20104d5 | |||
419d0912ea | |||
8c2f4f9823 | |||
4e6daa7065 | |||
ac77de69c2 | |||
7b2a3f3333 | |||
a904d1db61 | |||
6a290f7d5d | |||
97c6242097 | |||
5782ee83ad | |||
76503fda9a | |||
d36410f546 | |||
650217c0c5 | |||
a2a25205ce | |||
1aee3f97a3 | |||
c5c27dbc86 | |||
08e2f20817 | |||
8d110786c7 | |||
724a5425db | |||
ac0edcd804 | |||
d6404ce293 | |||
cb88bde406 | |||
f26018a9c0 | |||
78451d099c | |||
5570e66705 | |||
1626f2bd94 | |||
b56f6f5290 | |||
f1b2fc708e | |||
e697c2427a | |||
ffc5a07c17 | |||
fb9e2e0726 | |||
472a3ede4f | |||
c4655a9524 | |||
01dbfeed86 | |||
71abe3bf92 | |||
dbe755859c | |||
140f002bf8 | |||
4419fdcf2f | |||
e42fe74773 | |||
01e34dbcda | |||
59f1813dc5 | |||
5c9eb959d4 | |||
4479e05486 | |||
3b3229a7a5 | |||
6648f6197c | |||
64e94ebcda | |||
969dc8fc43 | |||
d6a52caa2e | |||
5284b3de3c | |||
d90ca1e16e | |||
6387738e0c | |||
391fb931f5 | |||
f680453377 | |||
c772a9ccc6 | |||
38adc9c883 | |||
f33e806949 | |||
f4ef9036d9 | |||
a64489f732 | |||
eafaf70f2d | |||
6b29c7a053 | |||
85f33ddfab | |||
d18b077279 | |||
f6938e6c8d | |||
0ad9a0988b | |||
235ea57a8f | |||
a6ed001163 | |||
3a7156a1b4 | |||
481f8ba892 | |||
06edb377a6 | |||
c2f79a722b | |||
d6eac69936 | |||
72684341c8 | |||
a926e2b3f3 | |||
07db5389c9 | |||
328c000029 | |||
3a79f8d45a | |||
43c645b597 | |||
88e5fc6380 | |||
6191de56c9 | |||
5423639e7b | |||
c225631afd | |||
b81d829cdb | |||
5b682fd40b | |||
3c6fb4a9a4 | |||
5acb063f68 | |||
08fdc050d7 | |||
b2bfd0f130 | |||
ef5f262671 | |||
a831c707bf | |||
9305be5e9f | |||
db74785593 | |||
4fbce4913d | |||
739eee4a65 | |||
bd7227e0c3 | |||
69982bc3b3 | |||
bda13fa868 | |||
c73fd3d91c | |||
deeca11946 | |||
ae667cb4ae | |||
419a536bfa | |||
1a17d0c27c | |||
4ed6524494 | |||
3edd9862e6 | |||
37d0148f78 | |||
90ccd72000 | |||
032fa5c5e4 | |||
b7bf60a5f7 | |||
ec5b0e19fc | |||
fd0768f916 | |||
3912a545cd | |||
6c4fff998e | |||
2856371066 | |||
beaa3476df | |||
05a61b80ab | |||
d5e3361cc5 | |||
c084df1245 | |||
954d4da58d | |||
4f42fc20b5 | |||
88a61333be | |||
42f1ace2dc | |||
e1a59558d0 | |||
60dc9d3648 | |||
5513777940 | |||
92e5575c87 | |||
6eab792f60 | |||
0445a18732 | |||
789454d92c | |||
0ae0217f51 | |||
565398b3aa | |||
2a9d271458 | |||
7dcc9e5fc3 | |||
422f985057 | |||
0db771b9ab | |||
5724191dd1 | |||
1ebc11120f | |||
7ee82f294a | |||
2de045bd7a | |||
fcb77afa4d | |||
0b542c7480 | |||
19d17f2de0 | |||
0335180c90 | |||
aae6024394 | |||
f016ec070e | |||
f8ba9f50be | |||
23255795fb | |||
1b63d697bd | |||
946a98c47e | |||
e837bc5a0c | |||
62870e168c | |||
421056684a | |||
451e84b610 | |||
1f5703370e | |||
7e8cc74dba | |||
bb2438c717 | |||
89600cb733 | |||
5c2b59ab34 | |||
c51d1e2f87 | |||
f3c064d740 | |||
6aefae24f7 | |||
537aa1903e | |||
f70e52dcc9 | |||
a304692ecb | |||
eeca7c5026 | |||
dfbcd2aee1 | |||
4421331437 | |||
5e6c1bb9e2 | |||
44eba43bae | |||
358a5e374f | |||
b9c65a53a1 | |||
9620ff9416 | |||
f973c7630e | |||
ac76264da2 | |||
b6b896c763 | |||
6d7c5c3914 | |||
df2c66cf29 | |||
f9d718302d | |||
c3ef0c3702 | |||
e498e7736a | |||
e1674c1af9 | |||
f233bcb89a | |||
4b237a63e5 | |||
af9f6d8b8e | |||
fb6358de51 | |||
226c5fce6f | |||
009d23987c | |||
6f887e018c | |||
5af63786ee | |||
128458b745 | |||
ca6bec4c75 | |||
042a8136fa | |||
827c06dc36 | |||
0105abc295 | |||
eb940f762b | |||
f9482e011a | |||
1e52a5d65c | |||
2a74a2a5c4 | |||
6ab5ae85b2 | |||
024aa36e22 | |||
f6cb2006aa | |||
b517f99ae2 | |||
9636c5bc97 | |||
03e4befc7a | |||
cdfc708aeb | |||
7d5614c3eb | |||
c675eb70db | |||
423fbafafc | |||
bcfc6f96bb | |||
d34ac2aef3 | |||
e4b538ed37 | |||
a288c0d3a6 | |||
b7a34f8320 | |||
129bcf9a1f | |||
153576c006 | |||
a9a9dff640 | |||
c5f5ea997d | |||
4a71b18a91 | |||
a4fdb6b72e | |||
ee1123f130 | |||
4ef1b2ec49 | |||
d218332866 | |||
f00c9148c4 | |||
f8f7d958bb | |||
bfc4fa1919 | |||
43be8d0cdd | |||
8067f9f84a | |||
f47759687a | |||
087e546081 | |||
5371bd590f | |||
24734036d2 | |||
21713123c6 | |||
9d1a680eed | |||
46a391f1c2 | |||
357a955e8e | |||
fce0498922 | |||
110e97717e | |||
1aecf21113 | |||
ef112b5be6 | |||
5125ed6d61 | |||
f05da40d4c | |||
a143bf8921 | |||
326fa87401 | |||
d2ac25d73f | |||
786003e9c7 | |||
62358f9da4 | |||
fe302dcadf | |||
c266032a10 | |||
d11c12cf90 | |||
959b4c2ca0 | |||
149c153930 | |||
1536a018c7 | |||
b436a39715 | |||
f8f6480e7d | |||
836191867d | |||
0559ce1d80 | |||
b88829864d | |||
b19e2b3c0a | |||
12240bc515 | |||
49b6acfed8 | |||
bab998e698 | |||
a832735f53 | |||
d36089f6c3 | |||
409e5d240e | |||
549764e76a | |||
f05cdc6faa | |||
a6989616e2 | |||
4f60444900 | |||
ac08543e75 | |||
75023663df | |||
4ff279027a | |||
31d144d116 | |||
15e20abff8 | |||
46f8c43497 | |||
14684a93ef | |||
0e9fa9c8e2 | |||
15ed67c1de | |||
caa1fa532d | |||
e62c1052cf | |||
caf7c34f5a | |||
7376bdf790 | |||
1f0d323737 | |||
a39c3ffed3 | |||
9a759fbc45 | |||
62e84c3c4c | |||
b25b8c7a74 | |||
bbd70929c2 | |||
60abaa8ea2 | |||
8c1e004312 | |||
6f6d8132f3 | |||
219bd32ab9 | |||
7ede1b4cda | |||
0117eff7bc | |||
06f6e1a8f1 | |||
df080e3b0c | |||
83f444df15 | |||
28e8ba14c9 | |||
087718c925 | |||
d1dab7c3b6 | |||
02697c5273 | |||
4deddfaddc | |||
932e5370e4 | |||
f6bac355f6 | |||
851656297a | |||
42fcadffd7 | |||
9110cf0ecc | |||
ea8f042b47 | |||
4603c8008d | |||
df58a55048 | |||
7cce37b0b4 | |||
fee05dc7f3 | |||
ce06d75c94 | |||
988235bf28 | |||
aa4608ff29 | |||
18265ba4d8 | |||
7bac37febb | |||
6dfbaa3535 | |||
54d492b16c | |||
27ed31ebd6 | |||
e027344f45 | |||
6d0159e91a | |||
4e7bbc596f | |||
6bfe1df230 | |||
8e3dd7cda3 | |||
d9309c3ecd | |||
27d36d540a | |||
15d9fe194e | |||
23ef4c9b80 | |||
ad3b558f83 | |||
b2f22d964b | |||
85a08b9006 | |||
2036e240da | |||
17d4bc397f | |||
c355eeab11 | |||
db69912a35 | |||
f14c093626 | |||
7f1a4282b7 | |||
0adb1986cf | |||
75ba1c2aad | |||
ac8bda106a | |||
1d44e6d1ee | |||
89fe568d08 | |||
c697566258 | |||
9ce9ed1016 | |||
3bb6112eb2 | |||
e4395de4fb | |||
52d77950b7 | |||
ebc78f9b2f | |||
835e616ef0 | |||
5c56cd6c9d | |||
e3c53feccf | |||
3b9f80ede5 | |||
9845713000 | |||
e789ee158f | |||
f2e9a96352 | |||
818081ab4c | |||
6671836543 | |||
526907ee72 | |||
6b6ba165e0 | |||
9cab4efcbd | |||
0c765721e5 | |||
0dfde21f77 | |||
9dcf389701 | |||
34d3904887 | |||
e30b22f33a | |||
8f30d2584c | |||
0b64a402d8 | |||
973b3a337d | |||
6ea3de1934 | |||
6eec6af39e |
73
.github/ISSUE_TEMPLATE/bug_report.yml
vendored
Normal file
73
.github/ISSUE_TEMPLATE/bug_report.yml
vendored
Normal file
@ -0,0 +1,73 @@
|
|||||||
|
name: Bug Report
|
||||||
|
description: "Create a report to help us improve."
|
||||||
|
body:
|
||||||
|
- type: checkboxes
|
||||||
|
id: terms
|
||||||
|
attributes:
|
||||||
|
label: Welcome
|
||||||
|
options:
|
||||||
|
- label: Yes, I understand that the GitHub action repository is not the repository of golangci-lint itself.
|
||||||
|
required: true
|
||||||
|
- label: Yes, I've searched similar issues on GitHub and didn't find any.
|
||||||
|
required: true
|
||||||
|
- label: Yes, I've included all information below (version, config, etc).
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: textarea
|
||||||
|
id: problem
|
||||||
|
attributes:
|
||||||
|
label: Description of the problem
|
||||||
|
placeholder: Your problem description
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: input
|
||||||
|
id: golangci-lint-version
|
||||||
|
attributes:
|
||||||
|
label: Version of golangci-lint
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: input
|
||||||
|
id: github-action-version
|
||||||
|
attributes:
|
||||||
|
label: Version of the GitHub Action
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: textarea
|
||||||
|
id: config
|
||||||
|
attributes:
|
||||||
|
label: Workflow file
|
||||||
|
value: |-
|
||||||
|
<details>
|
||||||
|
|
||||||
|
```
|
||||||
|
<add you file here>
|
||||||
|
```
|
||||||
|
|
||||||
|
</details>
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: input
|
||||||
|
id: go-env
|
||||||
|
attributes:
|
||||||
|
label: Go version
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: textarea
|
||||||
|
id: code-example
|
||||||
|
attributes:
|
||||||
|
label: Code example or link to a public repository
|
||||||
|
value: |-
|
||||||
|
<details>
|
||||||
|
|
||||||
|
```go
|
||||||
|
// add your code here
|
||||||
|
```
|
||||||
|
|
||||||
|
</details>
|
||||||
|
validations:
|
||||||
|
required: true
|
11
.github/ISSUE_TEMPLATE/config.yml
vendored
Normal file
11
.github/ISSUE_TEMPLATE/config.yml
vendored
Normal file
@ -0,0 +1,11 @@
|
|||||||
|
blank_issues_enabled: false
|
||||||
|
contact_links:
|
||||||
|
- name: Questions
|
||||||
|
url: https://github.com/golangci/golangci-lint-action/discussions
|
||||||
|
about: If you have a question, or are looking for advice, please post on our Discussions forum!
|
||||||
|
- name: golangci-lint main repository
|
||||||
|
url: https://github.com/golangci/golangci-lint
|
||||||
|
about: The main repository of golangci-lint.
|
||||||
|
- name: golangci-lint documentation
|
||||||
|
url: https://golangci-lint.run
|
||||||
|
about: Please take a look to our documentation.
|
44
.github/ISSUE_TEMPLATE/feature_request.yml
vendored
Normal file
44
.github/ISSUE_TEMPLATE/feature_request.yml
vendored
Normal file
@ -0,0 +1,44 @@
|
|||||||
|
name: Feature request
|
||||||
|
description: "Suggest an idea for this project."
|
||||||
|
body:
|
||||||
|
- type: checkboxes
|
||||||
|
id: terms
|
||||||
|
attributes:
|
||||||
|
label: Welcome
|
||||||
|
options:
|
||||||
|
- label: Yes, I understand that the GitHub action repository is not the repository of golangci-lint itself.
|
||||||
|
required: true
|
||||||
|
- label: Yes, I've searched similar issues on GitHub and didn't find any.
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: textarea
|
||||||
|
id: problem
|
||||||
|
attributes:
|
||||||
|
label: Your feature request related to a problem? Please describe.
|
||||||
|
placeholder: "A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]"
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: textarea
|
||||||
|
id: solution
|
||||||
|
attributes:
|
||||||
|
label: Describe the solution you'd like.
|
||||||
|
placeholder: "A clear and concise description of what you want to happen."
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: textarea
|
||||||
|
id: alternatives
|
||||||
|
attributes:
|
||||||
|
label: Describe alternatives you've considered.
|
||||||
|
placeholder: "A clear and concise description of any alternative solutions or features you've considered."
|
||||||
|
validations:
|
||||||
|
required: true
|
||||||
|
|
||||||
|
- type: textarea
|
||||||
|
id: additional
|
||||||
|
attributes:
|
||||||
|
label: Additional context.
|
||||||
|
placeholder: "Add any other context or screenshots about the feature request here."
|
||||||
|
validations:
|
||||||
|
required: false
|
4
.github/dependabot.yml
vendored
4
.github/dependabot.yml
vendored
@ -4,11 +4,7 @@ updates:
|
|||||||
directory: "/"
|
directory: "/"
|
||||||
schedule:
|
schedule:
|
||||||
interval: weekly
|
interval: weekly
|
||||||
reviewers:
|
|
||||||
- "golangci/team"
|
|
||||||
- package-ecosystem: npm
|
- package-ecosystem: npm
|
||||||
directory: "/"
|
directory: "/"
|
||||||
schedule:
|
schedule:
|
||||||
interval: weekly
|
interval: weekly
|
||||||
reviewers:
|
|
||||||
- "golangci/team"
|
|
||||||
|
6
.github/workflows/codeql.yaml
vendored
6
.github/workflows/codeql.yaml
vendored
@ -16,7 +16,7 @@ jobs:
|
|||||||
|
|
||||||
steps:
|
steps:
|
||||||
- name: Checkout repository
|
- name: Checkout repository
|
||||||
uses: actions/checkout@v2
|
uses: actions/checkout@v3
|
||||||
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.
|
||||||
@ -31,7 +31,7 @@ jobs:
|
|||||||
|
|
||||||
# Initializes the CodeQL tools for scanning.
|
# Initializes the CodeQL tools for scanning.
|
||||||
- name: Initialize CodeQL
|
- name: Initialize CodeQL
|
||||||
uses: github/codeql-action/init@v1
|
uses: github/codeql-action/init@v2
|
||||||
# Override language selection by uncommenting this and choosing your languages
|
# Override language selection by uncommenting this and choosing your languages
|
||||||
with:
|
with:
|
||||||
language: 'javascript'
|
language: 'javascript'
|
||||||
@ -41,4 +41,4 @@ jobs:
|
|||||||
npm run all
|
npm run all
|
||||||
|
|
||||||
- name: Perform CodeQL Analysis
|
- name: Perform CodeQL Analysis
|
||||||
uses: github/codeql-action/analyze@v1
|
uses: github/codeql-action/analyze@v2
|
||||||
|
53
.github/workflows/test.yml
vendored
53
.github/workflows/test.yml
vendored
@ -10,12 +10,39 @@ jobs:
|
|||||||
build: # make sure build/ci work properly
|
build: # make sure build/ci work properly
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v2
|
- if: ${{ !(github.event_name == 'pull_request' && (github.event.pull_request.user.id == 49699333 || contains(github.event.pull_request.labels.*.name, 'dependencies'))) }}
|
||||||
|
uses: actions/checkout@v3
|
||||||
|
- if: github.event_name == 'pull_request' && (github.event.pull_request.user.id == 49699333 || contains(github.event.pull_request.labels.*.name, 'dependencies'))
|
||||||
|
uses: actions/checkout@v3
|
||||||
|
with:
|
||||||
|
fetch-depth: 0
|
||||||
|
ref: ${{ github.event.pull_request.head.ref }}
|
||||||
|
|
||||||
- run: |
|
- run: |
|
||||||
npm install
|
npm install
|
||||||
npm run all
|
npm run all
|
||||||
|
|
||||||
|
# Update dist files if there is label dependencies or pull request' author is dependabot[bot] (id = 49699333)
|
||||||
|
- name: Update dist files
|
||||||
|
if: github.event_name == 'pull_request' && (github.event.pull_request.user.id == 49699333 || contains(github.event.pull_request.labels.*.name, 'dependencies'))
|
||||||
|
run: |
|
||||||
|
if [[ -z $(git status -s) ]]
|
||||||
|
then
|
||||||
|
echo "No change is required"
|
||||||
|
else
|
||||||
|
echo "Updating dist directory"
|
||||||
|
git config --local user.name "dependabot[bot]"
|
||||||
|
git config --local user.email "49699333+dependabot[bot]@users.noreply.github.com"
|
||||||
|
git add --update
|
||||||
|
git commit --message="Update dist files"
|
||||||
|
git push
|
||||||
|
fi
|
||||||
|
|
||||||
|
# Fail the build if there is dirty change
|
||||||
|
- run: git diff --exit-code
|
||||||
|
|
||||||
test: # make sure the action works on a clean machine without building
|
test: # make sure the action works on a clean machine without building
|
||||||
|
needs: [ build ]
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
os:
|
os:
|
||||||
@ -25,18 +52,25 @@ jobs:
|
|||||||
version:
|
version:
|
||||||
- ""
|
- ""
|
||||||
- "latest"
|
- "latest"
|
||||||
- "v1.37"
|
- "v1.51"
|
||||||
- "v1.37.1"
|
- "v1.51.2"
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
|
pull-requests: read
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v2
|
- uses: actions/checkout@v3
|
||||||
|
- uses: actions/setup-go@v4
|
||||||
|
with:
|
||||||
|
cache: false # setup-go v4 caches by default
|
||||||
- uses: ./
|
- uses: ./
|
||||||
with:
|
with:
|
||||||
version: ${{ matrix.version }}
|
version: ${{ matrix.version }}
|
||||||
args: --issues-exit-code=0 ./sample/...
|
args: --timeout=3m --issues-exit-code=0 ./sample/...
|
||||||
only-new-issues: true
|
only-new-issues: true
|
||||||
|
|
||||||
test-go-mod-version:
|
test-go-mod-version:
|
||||||
|
needs: [ build ]
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
os:
|
os:
|
||||||
@ -44,9 +78,14 @@ jobs:
|
|||||||
- macos-latest
|
- macos-latest
|
||||||
- windows-latest
|
- windows-latest
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v2
|
- uses: actions/checkout@v3
|
||||||
|
- uses: actions/setup-go@v4
|
||||||
|
with:
|
||||||
|
cache: false # setup-go v4 caches by default
|
||||||
- uses: ./
|
- uses: ./
|
||||||
with:
|
with:
|
||||||
working-directory: sample-go-mod
|
working-directory: sample-go-mod
|
||||||
args: --issues-exit-code=0 ./...
|
args: --timeout=3m --issues-exit-code=0 ./...
|
||||||
|
39
README.md
39
README.md
@ -9,6 +9,8 @@ The action runs [golangci-lint](https://github.com/golangci/golangci-lint) and r
|
|||||||
|
|
||||||
## Compatibility
|
## Compatibility
|
||||||
|
|
||||||
|
* `v3.0.0+` requires explicit setup-go installation step prior to using this action: `uses: actions/setup-go@v3`.
|
||||||
|
The `skip-go-installation` option has been removed.
|
||||||
* `v2.0.0+` works with `golangci-lint` version >= `v1.28.3`
|
* `v2.0.0+` works with `golangci-lint` version >= `v1.28.3`
|
||||||
* `v1.2.2` is deprecated due to we forgot to change the minimum version of `golangci-lint` to `v1.28.3` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
|
* `v1.2.2` is deprecated due to we forgot to change the minimum version of `golangci-lint` to `v1.28.3` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
|
||||||
* `v1.2.1` works with `golangci-lint` version >= `v1.14.0` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
|
* `v1.2.1` works with `golangci-lint` version >= `v1.14.0` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
|
||||||
@ -27,14 +29,22 @@ on:
|
|||||||
- master
|
- master
|
||||||
- main
|
- main
|
||||||
pull_request:
|
pull_request:
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
|
# Optional: allow read access to pull request. Use with `only-new-issues` option.
|
||||||
|
# pull-requests: read
|
||||||
jobs:
|
jobs:
|
||||||
golangci:
|
golangci:
|
||||||
name: lint
|
name: lint
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v2
|
- uses: actions/setup-go@v4
|
||||||
|
with:
|
||||||
|
go-version: '1.17'
|
||||||
|
cache: false
|
||||||
|
- uses: actions/checkout@v3
|
||||||
- name: golangci-lint
|
- name: golangci-lint
|
||||||
uses: golangci/golangci-lint-action@v2
|
uses: golangci/golangci-lint-action@v3
|
||||||
with:
|
with:
|
||||||
# Optional: version of golangci-lint to use in form of v1.2 or v1.2.3 or `latest` to use the latest version
|
# Optional: version of golangci-lint to use in form of v1.2 or v1.2.3 or `latest` to use the latest version
|
||||||
version: v1.29
|
version: v1.29
|
||||||
@ -48,8 +58,9 @@ jobs:
|
|||||||
# Optional: show only new issues if it's a pull request. The default value is `false`.
|
# Optional: show only new issues if it's a pull request. The default value is `false`.
|
||||||
# only-new-issues: true
|
# only-new-issues: true
|
||||||
|
|
||||||
# Optional: if set to true then the action will use pre-installed Go.
|
# Optional: if set to true then the all caching functionality will be complete disabled,
|
||||||
# skip-go-installation: true
|
# takes precedence over all other caching options.
|
||||||
|
# skip-cache: true
|
||||||
|
|
||||||
# Optional: if set to true then the action don't cache or restore ~/go/pkg.
|
# Optional: if set to true then the action don't cache or restore ~/go/pkg.
|
||||||
# skip-pkg-cache: true
|
# skip-pkg-cache: true
|
||||||
@ -75,18 +86,26 @@ on:
|
|||||||
- master
|
- master
|
||||||
- main
|
- main
|
||||||
pull_request:
|
pull_request:
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
|
# Optional: allow read access to pull request. Use with `only-new-issues` option.
|
||||||
|
# pull-requests: read
|
||||||
jobs:
|
jobs:
|
||||||
golangci:
|
golangci:
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
go-version: [1.15.x]
|
go: [1.17]
|
||||||
os: [macos-latest, windows-latest]
|
os: [macos-latest, windows-latest]
|
||||||
name: lint
|
name: lint
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v2
|
- uses: actions/checkout@v3
|
||||||
|
- uses: actions/setup-go@v4
|
||||||
|
with:
|
||||||
|
go-version: ${{ matrix.go }}
|
||||||
|
cache: false
|
||||||
- name: golangci-lint
|
- name: golangci-lint
|
||||||
uses: golangci/golangci-lint-action@v2
|
uses: golangci/golangci-lint-action@v3
|
||||||
with:
|
with:
|
||||||
# Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
|
# Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
|
||||||
version: v1.29
|
version: v1.29
|
||||||
@ -125,7 +144,6 @@ The action was implemented with performance in mind:
|
|||||||
|
|
||||||
For example, in a repository of [golangci-lint](https://github.com/golangci/golangci-lint) running this action without the cache takes 50s, but with cache takes 14s:
|
For example, in a repository of [golangci-lint](https://github.com/golangci/golangci-lint) running this action without the cache takes 50s, but with cache takes 14s:
|
||||||
* in parallel:
|
* in parallel:
|
||||||
* 13s to download Go
|
|
||||||
* 4s to restore 50 MB of cache
|
* 4s to restore 50 MB of cache
|
||||||
* 1s to find and install `golangci-lint`
|
* 1s to find and install `golangci-lint`
|
||||||
* 1s to run `golangci-lint` (it takes 35s without cache)
|
* 1s to run `golangci-lint` (it takes 35s without cache)
|
||||||
@ -145,16 +163,15 @@ Inside our action we perform 3 steps:
|
|||||||
* restore [cache](https://github.com/actions/cache) of previous analyzes
|
* restore [cache](https://github.com/actions/cache) of previous analyzes
|
||||||
* fetch [action config](https://github.com/golangci/golangci-lint/blob/master/assets/github-action-config.json) and find the latest `golangci-lint` patch version
|
* fetch [action config](https://github.com/golangci/golangci-lint/blob/master/assets/github-action-config.json) and find the latest `golangci-lint` patch version
|
||||||
for needed version (users of this action can specify only minor version of `golangci-lint`). After that install [golangci-lint](https://github.com/golangci/golangci-lint) using [@actions/tool-cache](https://github.com/actions/toolkit/tree/master/packages/tool-cache)
|
for needed version (users of this action can specify only minor version of `golangci-lint`). After that install [golangci-lint](https://github.com/golangci/golangci-lint) using [@actions/tool-cache](https://github.com/actions/toolkit/tree/master/packages/tool-cache)
|
||||||
* install the latest Go 1.x version using [@actions/setup-go](https://github.com/actions/setup-go)
|
|
||||||
2. Run `golangci-lint` with specified by user `args`
|
2. Run `golangci-lint` with specified by user `args`
|
||||||
3. Save cache for later builds
|
3. Save cache for later builds
|
||||||
|
|
||||||
### Caching internals
|
### Caching internals
|
||||||
|
|
||||||
1. We save and restore the following directories: `~/.cache/golangci-lint`, `~/.cache/go-build`, `~/go/pkg`.
|
1. We save and restore the following directories: `~/.cache/golangci-lint`, `~/.cache/go-build`, `~/go/pkg`.
|
||||||
2. The primary caching key looks like `golangci-lint.cache-{platform-arch}-{interval_number}-{go.mod_hash}`. Interval number ensures that we periodically invalidate
|
2. The primary caching key looks like `golangci-lint.cache-{interval_number}-{go.mod_hash}`. Interval number ensures that we periodically invalidate
|
||||||
our cache (every 7 days). `go.mod` hash ensures that we invalidate the cache early - as soon as dependencies have changed.
|
our cache (every 7 days). `go.mod` hash ensures that we invalidate the cache early - as soon as dependencies have changed.
|
||||||
3. We use [restore keys](https://help.github.com/en/actions/configuring-and-managing-workflows/caching-dependencies-to-speed-up-workflows#matching-a-cache-key): `golangci-lint.cache-{interval_number}-`, `golangci-lint.cache-`. GitHub matches keys by prefix if we have no exact match for the primary cache.
|
3. We use [restore keys](https://help.github.com/en/actions/configuring-and-managing-workflows/caching-dependencies-to-speed-up-workflows#matching-a-cache-key): `golangci-lint.cache-{interval_number}-`. GitHub matches keys by prefix if we have no exact match for the primary cache.
|
||||||
|
|
||||||
This scheme is basic and needs improvements. Pull requests and ideas are welcome.
|
This scheme is basic and needs improvements. Pull requests and ideas are welcome.
|
||||||
|
|
||||||
|
22
action.yml
22
action.yml
@ -15,25 +15,27 @@ inputs:
|
|||||||
github-token:
|
github-token:
|
||||||
description: "the token is used for fetching patch of a pull request to show only new issues"
|
description: "the token is used for fetching patch of a pull request to show only new issues"
|
||||||
default: ${{ github.token }}
|
default: ${{ github.token }}
|
||||||
required: true
|
required: false
|
||||||
only-new-issues:
|
only-new-issues:
|
||||||
description: "if set to true and the action runs on a pull request - the action outputs only newly found issues"
|
description: "if set to true and the action runs on a pull request - the action outputs only newly found issues"
|
||||||
default: false
|
default: false
|
||||||
required: true
|
required: false
|
||||||
skip-go-installation:
|
skip-cache:
|
||||||
description: "if set to true then action uses pre-installed Go"
|
description: |
|
||||||
|
if set to true then the all caching functionality will be complete disabled,
|
||||||
|
takes precedence over all other caching options.
|
||||||
default: false
|
default: false
|
||||||
required: true
|
required: false
|
||||||
skip-pkg-cache:
|
skip-pkg-cache:
|
||||||
description: "if set to true then the action don't cache or restore ~/go/pkg."
|
description: "if set to true then the action doesn't cache or restore ~/go/pkg."
|
||||||
default: false
|
default: false
|
||||||
required: true
|
required: false
|
||||||
skip-build-cache:
|
skip-build-cache:
|
||||||
description: "if set to true then the action don't cache or restore ~/.cache/go-build."
|
description: "if set to true then the action doesn't cache or restore ~/.cache/go-build."
|
||||||
default: false
|
default: false
|
||||||
required: true
|
required: false
|
||||||
runs:
|
runs:
|
||||||
using: "node12"
|
using: "node16"
|
||||||
main: "dist/run/index.js"
|
main: "dist/run/index.js"
|
||||||
post: "dist/post_run/index.js"
|
post: "dist/post_run/index.js"
|
||||||
branding:
|
branding:
|
||||||
|
116018
dist/post_run/index.js
vendored
116018
dist/post_run/index.js
vendored
File diff suppressed because one or more lines are too long
116026
dist/run/index.js
vendored
116026
dist/run/index.js
vendored
File diff suppressed because one or more lines are too long
5175
package-lock.json
generated
5175
package-lock.json
generated
File diff suppressed because it is too large
Load Diff
45
package.json
45
package.json
@ -1,12 +1,10 @@
|
|||||||
{
|
{
|
||||||
"name": "golanci-lint-action",
|
"name": "golanci-lint-action",
|
||||||
"version": "2.0.0",
|
"version": "3.1.0",
|
||||||
"private": true,
|
"private": true,
|
||||||
"description": "golangci-lint github action",
|
"description": "golangci-lint github action",
|
||||||
"main": "dist/main.js",
|
"main": "dist/main.js",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
"prepare-setup-go": "cd node_modules/setup-go && npm run build",
|
|
||||||
"prepare-deps": "npm run prepare-setup-go",
|
|
||||||
"build": "tsc && ncc build -o dist/run/ src/main.ts && ncc build -o dist/post_run/ src/post_main.ts",
|
"build": "tsc && ncc build -o dist/run/ src/main.ts && ncc build -o dist/post_run/ src/post_main.ts",
|
||||||
"watched_build_main": "tsc && ncc build -w -o dist/run/ src/main.ts",
|
"watched_build_main": "tsc && ncc build -w -o dist/run/ src/main.ts",
|
||||||
"watched_build_post_main": "tsc && ncc build -w -o dist/post_run/ src/post_main.ts",
|
"watched_build_post_main": "tsc && ncc build -w -o dist/post_run/ src/post_main.ts",
|
||||||
@ -15,7 +13,7 @@
|
|||||||
"lint-fix": "eslint **/*.ts --cache --fix",
|
"lint-fix": "eslint **/*.ts --cache --fix",
|
||||||
"format": "prettier --write **/*.ts",
|
"format": "prettier --write **/*.ts",
|
||||||
"format-check": "prettier --check **/*.ts",
|
"format-check": "prettier --check **/*.ts",
|
||||||
"all": "npm run prepare-deps && npm run build && npm run format-check && npm run lint",
|
"all": "npm run build && npm run format-check && npm run lint",
|
||||||
"local": "npm run build && act -j test -b",
|
"local": "npm run build && act -j test -b",
|
||||||
"local-full-version": "npm run build && act -j test-full-version -b"
|
"local-full-version": "npm run build && act -j test-full-version -b"
|
||||||
},
|
},
|
||||||
@ -26,28 +24,27 @@
|
|||||||
"author": "golangci",
|
"author": "golangci",
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"@actions/cache": "^1.0.6",
|
"@actions/cache": "^3.2.1",
|
||||||
"@actions/core": "^1.2.6",
|
"@actions/core": "^1.10.0",
|
||||||
"@actions/exec": "^1.0.1",
|
"@actions/exec": "^1.1.1",
|
||||||
"@actions/github": "^4.0.0",
|
"@actions/github": "^5.1.1",
|
||||||
"@actions/tool-cache": "^1.6.1",
|
"@actions/http-client": "^2.1.0",
|
||||||
"@types/semver": "^7.3.4",
|
"@actions/tool-cache": "^2.0.1",
|
||||||
"@types/tmp": "^0.2.0",
|
"@types/node": "^20.2.5",
|
||||||
"setup-go": "git+https://github.com/actions/setup-go.git#v2.1.3",
|
"@types/semver": "^7.5.0",
|
||||||
|
"@types/tmp": "^0.2.3",
|
||||||
"tmp": "^0.2.1"
|
"tmp": "^0.2.1"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"@types/node": "^14.14.31",
|
"@typescript-eslint/eslint-plugin": "^5.59.7",
|
||||||
"@types/uuid": "^8.3.0",
|
"@typescript-eslint/parser": "^5.59.7",
|
||||||
"@typescript-eslint/eslint-plugin": "^4.15.1",
|
"@vercel/ncc": "^0.36.1",
|
||||||
"@typescript-eslint/parser": "^4.15.1",
|
"eslint": "^8.41.0",
|
||||||
"@zeit/ncc": "^0.22.3",
|
"eslint-config-prettier": "^8.8.0",
|
||||||
"eslint": "^7.20.0",
|
"eslint-plugin-import": "^2.27.5",
|
||||||
"eslint-config-prettier": "^8.0.0",
|
"eslint-plugin-prettier": "^4.2.1",
|
||||||
"eslint-plugin-import": "^2.22.1",
|
"eslint-plugin-simple-import-sort": "^10.0.0",
|
||||||
"eslint-plugin-prettier": "^3.3.1",
|
"prettier": "^2.8.8",
|
||||||
"eslint-plugin-simple-import-sort": "^7.0.0",
|
"typescript": "^5.0.4"
|
||||||
"prettier": "^2.2.1",
|
|
||||||
"typescript": "^4.1.5"
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -1,5 +1,179 @@
|
|||||||
module sample
|
module sample
|
||||||
|
|
||||||
go 1.15
|
go 1.19
|
||||||
|
|
||||||
require github.com/golangci/golangci-lint v1.37.1
|
require github.com/golangci/golangci-lint v1.51.2
|
||||||
|
|
||||||
|
require (
|
||||||
|
4d63.com/gocheckcompilerdirectives v1.2.1 // indirect
|
||||||
|
4d63.com/gochecknoglobals v0.2.1 // indirect
|
||||||
|
github.com/Abirdcfly/dupword v0.0.9 // indirect
|
||||||
|
github.com/Antonboom/errname v0.1.7 // indirect
|
||||||
|
github.com/Antonboom/nilnil v0.1.1 // indirect
|
||||||
|
github.com/BurntSushi/toml v1.2.1 // indirect
|
||||||
|
github.com/Djarvur/go-err113 v0.0.0-20210108212216-aea10b59be24 // indirect
|
||||||
|
github.com/GaijinEntertainment/go-exhaustruct/v2 v2.3.0 // indirect
|
||||||
|
github.com/Masterminds/semver v1.5.0 // indirect
|
||||||
|
github.com/OpenPeeDeeP/depguard v1.1.1 // indirect
|
||||||
|
github.com/alexkohler/prealloc v1.0.0 // indirect
|
||||||
|
github.com/alingse/asasalint v0.0.11 // indirect
|
||||||
|
github.com/ashanbrown/forbidigo v1.4.0 // indirect
|
||||||
|
github.com/ashanbrown/makezero v1.1.1 // indirect
|
||||||
|
github.com/beorn7/perks v1.0.1 // indirect
|
||||||
|
github.com/bkielbasa/cyclop v1.2.0 // indirect
|
||||||
|
github.com/blizzy78/varnamelen v0.8.0 // indirect
|
||||||
|
github.com/bombsimon/wsl/v3 v3.4.0 // indirect
|
||||||
|
github.com/breml/bidichk v0.2.3 // indirect
|
||||||
|
github.com/breml/errchkjson v0.3.0 // indirect
|
||||||
|
github.com/butuzov/ireturn v0.1.1 // indirect
|
||||||
|
github.com/cespare/xxhash/v2 v2.1.2 // indirect
|
||||||
|
github.com/charithe/durationcheck v0.0.9 // indirect
|
||||||
|
github.com/chavacava/garif v0.0.0-20221024190013-b3ef35877348 // indirect
|
||||||
|
github.com/curioswitch/go-reassign v0.2.0 // indirect
|
||||||
|
github.com/daixiang0/gci v0.9.1 // indirect
|
||||||
|
github.com/davecgh/go-spew v1.1.1 // indirect
|
||||||
|
github.com/denis-tingaikin/go-header v0.4.3 // indirect
|
||||||
|
github.com/esimonov/ifshort v1.0.4 // indirect
|
||||||
|
github.com/ettle/strcase v0.1.1 // indirect
|
||||||
|
github.com/fatih/color v1.14.1 // indirect
|
||||||
|
github.com/fatih/structtag v1.2.0 // indirect
|
||||||
|
github.com/firefart/nonamedreturns v1.0.4 // indirect
|
||||||
|
github.com/fsnotify/fsnotify v1.5.4 // indirect
|
||||||
|
github.com/fzipp/gocyclo v0.6.0 // indirect
|
||||||
|
github.com/go-critic/go-critic v0.6.7 // indirect
|
||||||
|
github.com/go-toolsmith/astcast v1.1.0 // indirect
|
||||||
|
github.com/go-toolsmith/astcopy v1.0.3 // indirect
|
||||||
|
github.com/go-toolsmith/astequal v1.1.0 // indirect
|
||||||
|
github.com/go-toolsmith/astfmt v1.1.0 // indirect
|
||||||
|
github.com/go-toolsmith/astp v1.1.0 // indirect
|
||||||
|
github.com/go-toolsmith/strparse v1.1.0 // indirect
|
||||||
|
github.com/go-toolsmith/typep v1.1.0 // indirect
|
||||||
|
github.com/go-xmlfmt/xmlfmt v1.1.2 // indirect
|
||||||
|
github.com/gobwas/glob v0.2.3 // indirect
|
||||||
|
github.com/gofrs/flock v0.8.1 // indirect
|
||||||
|
github.com/golang/protobuf v1.5.2 // indirect
|
||||||
|
github.com/golangci/check v0.0.0-20180506172741-cfe4005ccda2 // indirect
|
||||||
|
github.com/golangci/dupl v0.0.0-20180902072040-3e9179ac440a // indirect
|
||||||
|
github.com/golangci/go-misc v0.0.0-20220329215616-d24fe342adfe // indirect
|
||||||
|
github.com/golangci/gofmt v0.0.0-20220901101216-f2edd75033f2 // indirect
|
||||||
|
github.com/golangci/lint-1 v0.0.0-20191013205115-297bf364a8e0 // indirect
|
||||||
|
github.com/golangci/maligned v0.0.0-20180506175553-b1d89398deca // indirect
|
||||||
|
github.com/golangci/misspell v0.4.0 // indirect
|
||||||
|
github.com/golangci/revgrep v0.0.0-20220804021717-745bb2f7c2e6 // indirect
|
||||||
|
github.com/golangci/unconvert v0.0.0-20180507085042-28b1c447d1f4 // indirect
|
||||||
|
github.com/google/go-cmp v0.5.9 // indirect
|
||||||
|
github.com/gordonklaus/ineffassign v0.0.0-20230107090616-13ace0543b28 // indirect
|
||||||
|
github.com/gostaticanalysis/analysisutil v0.7.1 // indirect
|
||||||
|
github.com/gostaticanalysis/comment v1.4.2 // indirect
|
||||||
|
github.com/gostaticanalysis/forcetypeassert v0.1.0 // indirect
|
||||||
|
github.com/gostaticanalysis/nilerr v0.1.1 // indirect
|
||||||
|
github.com/hashicorp/errwrap v1.0.0 // indirect
|
||||||
|
github.com/hashicorp/go-multierror v1.1.1 // indirect
|
||||||
|
github.com/hashicorp/go-version v1.6.0 // indirect
|
||||||
|
github.com/hashicorp/hcl v1.0.0 // indirect
|
||||||
|
github.com/hexops/gotextdiff v1.0.3 // indirect
|
||||||
|
github.com/inconshreveable/mousetrap v1.0.1 // indirect
|
||||||
|
github.com/jgautheron/goconst v1.5.1 // indirect
|
||||||
|
github.com/jingyugao/rowserrcheck v1.1.1 // indirect
|
||||||
|
github.com/jirfag/go-printf-func-name v0.0.0-20200119135958-7558a9eaa5af // indirect
|
||||||
|
github.com/julz/importas v0.1.0 // indirect
|
||||||
|
github.com/junk1tm/musttag v0.4.5 // indirect
|
||||||
|
github.com/kisielk/errcheck v1.6.3 // indirect
|
||||||
|
github.com/kisielk/gotool v1.0.0 // indirect
|
||||||
|
github.com/kkHAIKE/contextcheck v1.1.3 // indirect
|
||||||
|
github.com/kulti/thelper v0.6.3 // indirect
|
||||||
|
github.com/kunwardeep/paralleltest v1.0.6 // indirect
|
||||||
|
github.com/kyoh86/exportloopref v0.1.11 // indirect
|
||||||
|
github.com/ldez/gomoddirectives v0.2.3 // indirect
|
||||||
|
github.com/ldez/tagliatelle v0.4.0 // indirect
|
||||||
|
github.com/leonklingele/grouper v1.1.1 // indirect
|
||||||
|
github.com/lufeee/execinquery v1.2.1 // indirect
|
||||||
|
github.com/magiconair/properties v1.8.6 // indirect
|
||||||
|
github.com/maratori/testableexamples v1.0.0 // indirect
|
||||||
|
github.com/maratori/testpackage v1.1.0 // indirect
|
||||||
|
github.com/matoous/godox v0.0.0-20210227103229-6504466cf951 // indirect
|
||||||
|
github.com/mattn/go-colorable v0.1.13 // indirect
|
||||||
|
github.com/mattn/go-isatty v0.0.17 // indirect
|
||||||
|
github.com/mattn/go-runewidth v0.0.9 // indirect
|
||||||
|
github.com/matttproud/golang_protobuf_extensions v1.0.1 // indirect
|
||||||
|
github.com/mbilski/exhaustivestruct v1.2.0 // indirect
|
||||||
|
github.com/mgechev/revive v1.2.5 // indirect
|
||||||
|
github.com/mitchellh/go-homedir v1.1.0 // indirect
|
||||||
|
github.com/mitchellh/mapstructure v1.5.0 // indirect
|
||||||
|
github.com/moricho/tparallel v0.2.1 // indirect
|
||||||
|
github.com/nakabonne/nestif v0.3.1 // indirect
|
||||||
|
github.com/nbutton23/zxcvbn-go v0.0.0-20210217022336-fa2cb2858354 // indirect
|
||||||
|
github.com/nishanths/exhaustive v0.9.5 // indirect
|
||||||
|
github.com/nishanths/predeclared v0.2.2 // indirect
|
||||||
|
github.com/nunnatsa/ginkgolinter v0.8.1 // indirect
|
||||||
|
github.com/olekukonko/tablewriter v0.0.5 // indirect
|
||||||
|
github.com/pelletier/go-toml v1.9.5 // indirect
|
||||||
|
github.com/pelletier/go-toml/v2 v2.0.5 // indirect
|
||||||
|
github.com/pkg/errors v0.9.1 // indirect
|
||||||
|
github.com/pmezard/go-difflib v1.0.0 // indirect
|
||||||
|
github.com/polyfloyd/go-errorlint v1.1.0 // indirect
|
||||||
|
github.com/prometheus/client_golang v1.12.1 // indirect
|
||||||
|
github.com/prometheus/client_model v0.2.0 // indirect
|
||||||
|
github.com/prometheus/common v0.32.1 // indirect
|
||||||
|
github.com/prometheus/procfs v0.7.3 // indirect
|
||||||
|
github.com/quasilyte/go-ruleguard v0.3.19 // indirect
|
||||||
|
github.com/quasilyte/gogrep v0.5.0 // indirect
|
||||||
|
github.com/quasilyte/regex/syntax v0.0.0-20200407221936-30656e2c4a95 // indirect
|
||||||
|
github.com/quasilyte/stdinfo v0.0.0-20220114132959-f7386bf02567 // indirect
|
||||||
|
github.com/ryancurrah/gomodguard v1.3.0 // indirect
|
||||||
|
github.com/ryanrolds/sqlclosecheck v0.4.0 // indirect
|
||||||
|
github.com/sanposhiho/wastedassign/v2 v2.0.7 // indirect
|
||||||
|
github.com/sashamelentyev/interfacebloat v1.1.0 // indirect
|
||||||
|
github.com/sashamelentyev/usestdlibvars v1.23.0 // indirect
|
||||||
|
github.com/securego/gosec/v2 v2.15.0 // indirect
|
||||||
|
github.com/shazow/go-diff v0.0.0-20160112020656-b6b7b6733b8c // indirect
|
||||||
|
github.com/sirupsen/logrus v1.9.0 // indirect
|
||||||
|
github.com/sivchari/containedctx v1.0.2 // indirect
|
||||||
|
github.com/sivchari/nosnakecase v1.7.0 // indirect
|
||||||
|
github.com/sivchari/tenv v1.7.1 // indirect
|
||||||
|
github.com/sonatard/noctx v0.0.1 // indirect
|
||||||
|
github.com/sourcegraph/go-diff v0.7.0 // indirect
|
||||||
|
github.com/spf13/afero v1.8.2 // indirect
|
||||||
|
github.com/spf13/cast v1.5.0 // indirect
|
||||||
|
github.com/spf13/cobra v1.6.1 // indirect
|
||||||
|
github.com/spf13/jwalterweatherman v1.1.0 // indirect
|
||||||
|
github.com/spf13/pflag v1.0.5 // indirect
|
||||||
|
github.com/spf13/viper v1.12.0 // indirect
|
||||||
|
github.com/ssgreg/nlreturn/v2 v2.2.1 // indirect
|
||||||
|
github.com/stbenjam/no-sprintf-host-port v0.1.1 // indirect
|
||||||
|
github.com/stretchr/objx v0.5.0 // indirect
|
||||||
|
github.com/stretchr/testify v1.8.1 // indirect
|
||||||
|
github.com/subosito/gotenv v1.4.1 // indirect
|
||||||
|
github.com/t-yuki/gocover-cobertura v0.0.0-20180217150009-aaee18c8195c // indirect
|
||||||
|
github.com/tdakkota/asciicheck v0.1.1 // indirect
|
||||||
|
github.com/tetafro/godot v1.4.11 // indirect
|
||||||
|
github.com/timakin/bodyclose v0.0.0-20221125081123-e39cf3fc478e // indirect
|
||||||
|
github.com/timonwong/loggercheck v0.9.3 // indirect
|
||||||
|
github.com/tomarrell/wrapcheck/v2 v2.8.0 // indirect
|
||||||
|
github.com/tommy-muehle/go-mnd/v2 v2.5.1 // indirect
|
||||||
|
github.com/ultraware/funlen v0.0.3 // indirect
|
||||||
|
github.com/ultraware/whitespace v0.0.5 // indirect
|
||||||
|
github.com/uudashr/gocognit v1.0.6 // indirect
|
||||||
|
github.com/yagipy/maintidx v1.0.0 // indirect
|
||||||
|
github.com/yeya24/promlinter v0.2.0 // indirect
|
||||||
|
gitlab.com/bosi/decorder v0.2.3 // indirect
|
||||||
|
go.uber.org/atomic v1.7.0 // indirect
|
||||||
|
go.uber.org/multierr v1.6.0 // indirect
|
||||||
|
go.uber.org/zap v1.17.0 // indirect
|
||||||
|
golang.org/x/exp v0.0.0-20220722155223-a9213eeb770e // indirect
|
||||||
|
golang.org/x/exp/typeparams v0.0.0-20230203172020-98cc5a0785f9 // indirect
|
||||||
|
golang.org/x/mod v0.8.0 // indirect
|
||||||
|
golang.org/x/sync v0.1.0 // indirect
|
||||||
|
golang.org/x/sys v0.5.0 // indirect
|
||||||
|
golang.org/x/text v0.6.0 // indirect
|
||||||
|
golang.org/x/tools v0.6.0 // indirect
|
||||||
|
google.golang.org/protobuf v1.28.0 // indirect
|
||||||
|
gopkg.in/ini.v1 v1.67.0 // indirect
|
||||||
|
gopkg.in/yaml.v2 v2.4.0 // indirect
|
||||||
|
gopkg.in/yaml.v3 v3.0.1 // indirect
|
||||||
|
honnef.co/go/tools v0.4.2 // indirect
|
||||||
|
mvdan.cc/gofumpt v0.4.0 // indirect
|
||||||
|
mvdan.cc/interfacer v0.0.0-20180901003855-c20040233aed // indirect
|
||||||
|
mvdan.cc/lint v0.0.0-20170908181259-adc824a0674b // indirect
|
||||||
|
mvdan.cc/unparam v0.0.0-20221223090309-7455f1af531d // indirect
|
||||||
|
)
|
||||||
|
1019
sample-go-mod/go.sum
1019
sample-go-mod/go.sum
File diff suppressed because it is too large
Load Diff
19
src/cache.ts
19
src/cache.ts
@ -52,17 +52,18 @@ const getIntervalKey = (invalidationIntervalDays: number): string => {
|
|||||||
|
|
||||||
async function buildCacheKeys(): Promise<string[]> {
|
async function buildCacheKeys(): Promise<string[]> {
|
||||||
const keys = []
|
const keys = []
|
||||||
let cacheKey = `golangci-lint.cache-`
|
|
||||||
keys.push(cacheKey)
|
|
||||||
|
|
||||||
// Periodically invalidate a cache because a new code being added.
|
// Periodically invalidate a cache because a new code being added.
|
||||||
// TODO: configure it via inputs.
|
// TODO: configure it via inputs.
|
||||||
cacheKey += `${getIntervalKey(7)}-`
|
let cacheKey = `golangci-lint.cache-${getIntervalKey(7)}-`
|
||||||
keys.push(cacheKey)
|
keys.push(cacheKey)
|
||||||
|
// Get working directory from input
|
||||||
if (await pathExists(`go.mod`)) {
|
const workingDirectory = core.getInput(`working-directory`)
|
||||||
|
// create path to go.mod prepending the workingDirectory if it exists
|
||||||
|
const goModPath = path.join(workingDirectory, `go.mod`)
|
||||||
|
core.info(`Checking for go.mod: ${goModPath}`)
|
||||||
|
if (await pathExists(goModPath)) {
|
||||||
// Add checksum to key to invalidate a cache when dependencies change.
|
// Add checksum to key to invalidate a cache when dependencies change.
|
||||||
cacheKey += await checksumFile(`sha1`, `go.mod`)
|
cacheKey += await checksumFile(`sha1`, goModPath)
|
||||||
} else {
|
} else {
|
||||||
cacheKey += `nogomod`
|
cacheKey += `nogomod`
|
||||||
}
|
}
|
||||||
@ -72,6 +73,8 @@ async function buildCacheKeys(): Promise<string[]> {
|
|||||||
}
|
}
|
||||||
|
|
||||||
export async function restoreCache(): Promise<void> {
|
export async function restoreCache(): Promise<void> {
|
||||||
|
if (core.getInput(`skip-cache`, { required: true }).trim() == "true") return
|
||||||
|
|
||||||
if (!utils.isValidEvent()) {
|
if (!utils.isValidEvent()) {
|
||||||
utils.logWarning(
|
utils.logWarning(
|
||||||
`Event Validation Error: The event type ${process.env[Events.Key]} is not supported because it's not tied to a branch or tag ref.`
|
`Event Validation Error: The event type ${process.env[Events.Key]} is not supported because it's not tied to a branch or tag ref.`
|
||||||
@ -112,6 +115,8 @@ export async function restoreCache(): Promise<void> {
|
|||||||
}
|
}
|
||||||
|
|
||||||
export async function saveCache(): Promise<void> {
|
export async function saveCache(): Promise<void> {
|
||||||
|
if (core.getInput(`skip-cache`, { required: true }).trim() == "true") return
|
||||||
|
|
||||||
// Validate inputs, this can cause task failure
|
// Validate inputs, this can cause task failure
|
||||||
if (!utils.isValidEvent()) {
|
if (!utils.isValidEvent()) {
|
||||||
utils.logWarning(
|
utils.logWarning(
|
||||||
|
4
src/deps.d.ts
vendored
4
src/deps.d.ts
vendored
@ -1,7 +1,3 @@
|
|||||||
declare module "setup-go/lib/main" {
|
|
||||||
function run(): Promise<void>
|
|
||||||
}
|
|
||||||
|
|
||||||
declare module "cache/lib/restore" {
|
declare module "cache/lib/restore" {
|
||||||
function run(): Promise<void>
|
function run(): Promise<void>
|
||||||
export default run
|
export default run
|
||||||
|
@ -2,7 +2,6 @@ import * as core from "@actions/core"
|
|||||||
import * as tc from "@actions/tool-cache"
|
import * as tc from "@actions/tool-cache"
|
||||||
import os from "os"
|
import os from "os"
|
||||||
import path from "path"
|
import path from "path"
|
||||||
import { run as setupGo } from "setup-go/lib/main"
|
|
||||||
|
|
||||||
import { VersionConfig } from "./version"
|
import { VersionConfig } from "./version"
|
||||||
|
|
||||||
@ -59,16 +58,3 @@ export async function installLint(versionConfig: VersionConfig): Promise<string>
|
|||||||
core.info(`Installed golangci-lint into ${lintPath} in ${Date.now() - startedAt}ms`)
|
core.info(`Installed golangci-lint into ${lintPath} in ${Date.now() - startedAt}ms`)
|
||||||
return lintPath
|
return lintPath
|
||||||
}
|
}
|
||||||
|
|
||||||
export async function installGo(): Promise<void> {
|
|
||||||
const skipGoInstallation = core.getInput(`skip-go-installation`, { required: true }).trim()
|
|
||||||
if (skipGoInstallation.toLowerCase() == "true") {
|
|
||||||
core.info(`Skipping the installation of Go`)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
const startedAt = Date.now()
|
|
||||||
process.env[`INPUT_GO-VERSION`] = `1`
|
|
||||||
await setupGo()
|
|
||||||
core.info(`Installed Go in ${Date.now() - startedAt}ms`)
|
|
||||||
}
|
|
||||||
|
17
src/run.ts
17
src/run.ts
@ -7,7 +7,7 @@ import { dir } from "tmp"
|
|||||||
import { promisify } from "util"
|
import { promisify } from "util"
|
||||||
|
|
||||||
import { restoreCache, saveCache } from "./cache"
|
import { restoreCache, saveCache } from "./cache"
|
||||||
import { installGo, installLint } from "./install"
|
import { installLint } from "./install"
|
||||||
import { findLintVersion } from "./version"
|
import { findLintVersion } from "./version"
|
||||||
|
|
||||||
const execShellCommand = promisify(exec)
|
const execShellCommand = promisify(exec)
|
||||||
@ -41,7 +41,7 @@ async function fetchPatch(): Promise<string> {
|
|||||||
const octokit = github.getOctokit(core.getInput(`github-token`, { required: true }))
|
const octokit = github.getOctokit(core.getInput(`github-token`, { required: true }))
|
||||||
let patch: string
|
let patch: string
|
||||||
try {
|
try {
|
||||||
const patchResp = await octokit.pulls.get({
|
const patchResp = await octokit.rest.pulls.get({
|
||||||
owner: ctx.repo.owner,
|
owner: ctx.repo.owner,
|
||||||
repo: ctx.repo.repo,
|
repo: ctx.repo.repo,
|
||||||
[`pull_number`]: pull.number,
|
[`pull_number`]: pull.number,
|
||||||
@ -85,11 +85,9 @@ async function prepareEnv(): Promise<Env> {
|
|||||||
// Prepare cache, lint and go in parallel.
|
// Prepare cache, lint and go in parallel.
|
||||||
const restoreCachePromise = restoreCache()
|
const restoreCachePromise = restoreCache()
|
||||||
const prepareLintPromise = prepareLint()
|
const prepareLintPromise = prepareLint()
|
||||||
const installGoPromise = installGo()
|
|
||||||
const patchPromise = fetchPatch()
|
const patchPromise = fetchPatch()
|
||||||
|
|
||||||
const lintPath = await prepareLintPromise
|
const lintPath = await prepareLintPromise
|
||||||
await installGoPromise
|
|
||||||
await restoreCachePromise
|
await restoreCachePromise
|
||||||
const patchPath = await patchPromise
|
const patchPath = await patchPromise
|
||||||
|
|
||||||
@ -121,15 +119,14 @@ async function runLint(lintPath: string, patchPath: string): Promise<void> {
|
|||||||
const userArgs = core.getInput(`args`)
|
const userArgs = core.getInput(`args`)
|
||||||
const addedArgs: string[] = []
|
const addedArgs: string[] = []
|
||||||
|
|
||||||
const userArgNames = new Set<string>()
|
const userArgNames = new Set<string>(
|
||||||
userArgs
|
userArgs
|
||||||
.split(/\s/)
|
.trim()
|
||||||
|
.split(/\s+/)
|
||||||
.map((arg) => arg.split(`=`)[0])
|
.map((arg) => arg.split(`=`)[0])
|
||||||
.filter((arg) => arg.startsWith(`-`))
|
.filter((arg) => arg.startsWith(`-`))
|
||||||
.forEach((arg) => {
|
.map((arg) => arg.replace(/^-+/, ``))
|
||||||
userArgNames.add(arg.replace(`-`, ``))
|
)
|
||||||
})
|
|
||||||
|
|
||||||
if (userArgNames.has(`out-format`)) {
|
if (userArgNames.has(`out-format`)) {
|
||||||
throw new Error(`please, don't change out-format for golangci-lint: it can be broken in a future`)
|
throw new Error(`please, don't change out-format for golangci-lint: it can be broken in a future`)
|
||||||
}
|
}
|
||||||
|
@ -8,7 +8,8 @@
|
|||||||
"noImplicitAny": true, /* Raise error on expressions and declarations with an implied 'any' type. */
|
"noImplicitAny": true, /* Raise error on expressions and declarations with an implied 'any' type. */
|
||||||
"esModuleInterop": true, /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */
|
"esModuleInterop": true, /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */
|
||||||
"tsBuildInfoFile": ".tsbuildinfo",
|
"tsBuildInfoFile": ".tsbuildinfo",
|
||||||
"incremental": true
|
"incremental": true,
|
||||||
|
"useUnknownInCatchVariables": false
|
||||||
},
|
},
|
||||||
"exclude": ["node_modules", "**/*.test.ts"]
|
"exclude": ["node_modules", "**/*.test.ts"]
|
||||||
}
|
}
|
Reference in New Issue
Block a user