Skip to content

Commit fe2a5dc

Browse files
arimxyerclaude
andauthored
ci: slim down push-to-main CI and remove redundancies (#75)
- Skip integration tests on push to main (PRs already run them) - Remove security job from CI (weekly security-scan.yml covers this) - Delete no-op pr-validation.yml workflow - Update build job to handle skipped dependencies Rulesets also updated: - Main branch: removed "Security Scan" from required checks - Release tags: reduced to Lint + Unit Tests + Build only Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent ef4affa commit fe2a5dc

2 files changed

Lines changed: 3 additions & 70 deletions

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ on:
2929

3030
permissions:
3131
contents: read
32-
security-events: write # For SARIF upload
3332

3433
jobs:
3534
check-changes:
@@ -147,6 +146,7 @@ jobs:
147146

148147
integration-tests:
149148
name: Integration Tests (${{ matrix.os }})
149+
if: github.event_name == 'pull_request'
150150
runs-on: ${{ matrix.os }}
151151
needs: check-changes
152152
strategy:
@@ -187,53 +187,11 @@ jobs:
187187
fi
188188
shell: bash
189189

190-
security:
191-
name: Security Scan
192-
runs-on: ubuntu-latest
193-
needs: check-changes
194-
195-
steps:
196-
- name: Skip if no code changes
197-
if: needs.check-changes.outputs.code-changed != 'true'
198-
run: |
199-
echo "No code changes detected, skipping security scan"
200-
exit 0
201-
202-
- name: Checkout code
203-
if: needs.check-changes.outputs.code-changed == 'true'
204-
uses: actions/checkout@v6
205-
206-
- name: Set up Go
207-
if: needs.check-changes.outputs.code-changed == 'true'
208-
uses: actions/setup-go@v6
209-
with:
210-
go-version: '1.25'
211-
cache: true
212-
cache-dependency-path: go.sum
213-
214-
- name: Run Gosec Security Scanner
215-
if: needs.check-changes.outputs.code-changed == 'true'
216-
uses: securego/gosec@master
217-
with:
218-
args: '-no-fail -fmt sarif -out results.sarif ./...'
219-
220-
- name: Fix SARIF format
221-
if: needs.check-changes.outputs.code-changed == 'true'
222-
run: |
223-
jq '.runs[].results[] |= if .fixes then .fixes |= map(select(.artifactChanges and (.artifactChanges | type == "array" and length > 0))) else . end' results.sarif > results-fixed.sarif
224-
mv results-fixed.sarif results.sarif
225-
226-
- name: Upload SARIF file
227-
if: needs.check-changes.outputs.code-changed == 'true'
228-
uses: github/codeql-action/upload-sarif@v3
229-
with:
230-
sarif_file: results.sarif
231-
continue-on-error: true
232-
233190
build:
234191
name: Build
235192
runs-on: ubuntu-latest
236-
needs: [check-changes, unit-tests, integration-tests, security]
193+
needs: [check-changes, unit-tests, integration-tests]
194+
if: ${{ !failure() && !cancelled() }}
237195

238196
steps:
239197
- name: Skip if no code changes

.github/workflows/pr-validation.yml

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

0 commit comments

Comments
 (0)