Commit 3b50515
authored
fix(ci): prevent spurious Dependabot PR failures from auto-approve and @types/vscode bumps (#143)
* fix(ci): stop broken auto-approve runs for Dependabot and ignore @types/vscode
- Remove dependabot[bot] from auto-approve.yml condition. That workflow
runs on pull_request (no secrets for Dependabot) and always failed the
create-github-app-token step, producing a permanent red 'auto-approve'
check on every Dependabot PR.
- Dependabot approval + auto-merge (patch/minor) is already handled by
the dedicated dependabot-auto-merge.yml on pull_request_target.
- Add @types/vscode to dependabot ignore for npm. vsce package enforces
that the @types/vscode version must not exceed engines.vscode. Bumps
produced failing 'build' jobs (package step).
- New Dependabot PRs will no longer have spurious failures. Majors still
require manual review per existing policy.
Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
* ci(security): pin Gitleaks version to avoid rate-limit 404s on install
The previous dynamic fetch of 'latest' via unauthenticated GitHub API
was brittle:
- Rate limits (60/hr unauth) often return error JSON instead of release.
- grep/cut produces empty GITLEAKS_VERSION.
- Download URL becomes invalid → 404 + tar failure.
This caused the 'Gitleaks secret detection' job to fail on PR #143
(even though the PR changes are unrelated to secrets scanning).
Fix: pin to a recent stable release (8.30.1) with comment for future
updates. This is consistent with how other tools/actions are handled
in the repo.
Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
---------
Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>1 parent bf2f270 commit 3b50515
3 files changed
Lines changed: 8 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
10 | 15 | | |
11 | 16 | | |
12 | 17 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
26 | 25 | | |
27 | 26 | | |
28 | 27 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
83 | 83 | | |
84 | 84 | | |
85 | 85 | | |
| 86 | + | |
| 87 | + | |
86 | 88 | | |
87 | | - | |
| 89 | + | |
88 | 90 | | |
89 | 91 | | |
90 | 92 | | |
| |||
0 commit comments