Skip to content

Commit 29a3e49

Browse files
feat: auto-fix license files on PRs and improve CI reliability
Changes: - Pin go-licenses version in CI for reproducibility (commit 5348b744) - Add GOROOT/PATH setup for 'Package does not have module info' fix - Update license-check.yml to auto-fix and push to PR branches - Add CI=true env var to use pinned go-licenses version - Add dependabot exclusion from auto-fix workflow - Remove unnecessary fetch-depth: 0 from checkout - Fix comment body indentation in PR comment - Add code-scanning exclusion for third-party files - Simplify licenses-check to regenerate and compare
1 parent 02b4394 commit 29a3e49

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

.github/workflows/license-check.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,14 @@ permissions:
1919
jobs:
2020
license-check:
2121
runs-on: ubuntu-latest
22-
# Don't run on forks (they can't push back) or dependabot (has its own token)
23-
if: github.event.pull_request.head.repo.full_name == github.repository
22+
# Don't run on forks (they can't push back) or dependabot
23+
if: github.event.pull_request.head.repo.full_name == github.repository && github.actor != 'dependabot[bot]'
2424

2525
steps:
2626
- name: Check out code
2727
uses: actions/checkout@v6
2828
with:
2929
ref: ${{ github.head_ref }}
30-
# Need full history for push
31-
fetch-depth: 0
3230

3331
- name: Set up Go
3432
uses: actions/setup-go@v6
@@ -39,6 +37,8 @@ jobs:
3937
# which causes go-licenses to raise "Package ... does not have module info" errors.
4038
# For more information, https://github.com/google/go-licenses/issues/244#issuecomment-1885098633
4139
- name: Regenerate licenses
40+
env:
41+
CI: "true"
4242
run: |
4343
export GOROOT=$(go env GOROOT)
4444
export PATH=${GOROOT}/bin:$PATH
@@ -76,12 +76,12 @@ jobs:
7676
issue_number: context.issue.number,
7777
body: `## 📜 License files updated
7878
79-
I noticed the third-party license files were out of date and pushed a fix to this PR.
79+
I noticed the third-party license files were out of date and pushed a fix to this PR.
8080

81-
**What changed:** Dependencies were added, removed, or updated, which requires regenerating the license documentation.
81+
**What changed:** Dependencies were added, removed, or updated, which requires regenerating the license documentation.
8282

83-
**What I did:** Ran \`./script/licenses\` and committed the result.
83+
**What I did:** Ran \`./script/licenses\` and committed the result.
8484

85-
Please pull the latest changes before pushing again.`
85+
Please pull the latest changes before pushing again.`
8686
})
8787

0 commit comments

Comments
 (0)