-
Notifications
You must be signed in to change notification settings - Fork 393
ci(repo): Remove Semgrep workflow #6949
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Semgrep scans for this project are managed from Semgrep SaaS directly now, the workflow file causes the scans to run twice.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughThe change removes the GitHub Actions workflow at .github/workflows/semgrep.yml that previously ran Semgrep CI on workflow_dispatch, pull_request, push to main (with path filters), and a scheduled cron, using the returntocorp/semgrep container and SEMGREP_APP_TOKEN. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant Dev as Developer
participant GH as GitHub (Events)
participant WF as Semgrep Workflow (removed)
participant SG as Semgrep CI
rect rgb(245,248,255)
note over GH,WF: Previous flow (now removed)
Dev->>GH: Push / Pull Request / Schedule / Manual dispatch
GH->>WF: Trigger semgrep job
WF->>SG: Run "semgrep ci" in container
SG-->>WF: Results
WF-->>GH: Job status
end
rect rgb(250,250,250)
note over Dev,GH: Current state
Dev->>GH: Push / Pull Request / Schedule
GH--x WF: No workflow present
note right of GH: No Semgrep job runs
end
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/elements
@clerk/clerk-expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/clerk-react
@clerk/react-router
@clerk/remix
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/themes
@clerk/types
@clerk/upgrade
@clerk/vue
commit: |
Description
Semgrep scans for this project are managed from Semgrep SaaS directly now, the workflow file causes the scans to run twice.
Checklist
pnpm test
runs as expected.pnpm build
runs as expected.Type of change
Summary by CodeRabbit