Skip to content

Delete .github/workflows/security-scan.yml#599

Merged
IEvangelist merged 3 commits intomainfrom
delete-workflow
Mar 24, 2026
Merged

Delete .github/workflows/security-scan.yml#599
IEvangelist merged 3 commits intomainfrom
delete-workflow

Conversation

@IEvangelist
Copy link
Copy Markdown
Member

Let's just remove this.

Copilot AI review requested due to automatic review settings March 23, 2026 22:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the reusable GitHub Actions workflow that performs dependency/vulnerability scanning and uploads SARIF results to GitHub code scanning.

Changes:

  • Deleted the reusable workflow .github/workflows/security-scan.yml.
Comments suppressed due to low confidence (1)

.github/workflows/security-scan.yml:1

  • Deleting this reusable workflow will break .github/workflows/ci.yml, which currently defines a security-scan job that uses: ./.github/workflows/security-scan.yml. Either keep this workflow file, or update ci.yml in the same PR to remove/replace that job (e.g., inline the steps or point to a different reusable workflow).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@IEvangelist IEvangelist enabled auto-merge (squash) March 23, 2026 23:20
@IEvangelist IEvangelist disabled auto-merge March 24, 2026 02:30
@IEvangelist IEvangelist enabled auto-merge (squash) March 24, 2026 02:30
auto-merge was automatically disabled March 24, 2026 14:23

Pull request was closed

@IEvangelist IEvangelist reopened this Mar 24, 2026
@IEvangelist IEvangelist enabled auto-merge (squash) March 24, 2026 14:53
@IEvangelist IEvangelist merged commit ae8ed0a into main Mar 24, 2026
5 checks passed
@IEvangelist IEvangelist deleted the delete-workflow branch March 24, 2026 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants