Skip to content

[VULN-649] ci: Remove deprecated scan workflow#247

Merged
theMickster merged 1 commit into
mainfrom
vuln-649/remove-scan-workflow
Jul 1, 2026
Merged

[VULN-649] ci: Remove deprecated scan workflow#247
theMickster merged 1 commit into
mainfrom
vuln-649/remove-scan-workflow

Conversation

@theMickster

Copy link
Copy Markdown
Contributor

🎟️ Tracking

https://bitwarden.atlassian.net/browse/VULN-649

📔 Objective

Removes the deprecated scan.yml GitHub Actions workflow.

@theMickster theMickster added the ai-review Request a Claude code review label Jun 22, 2026
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR removes the deprecated .github/workflows/scan.yml workflow (Checkmarx SAST and Sonar quality jobs), tracked by VULN-649. I verified there are no orphaned references to the deleted workflow: the retained examples/scan.yaml is an independent lint fixture (a starter-workflow template exercised by the linter), and the check-run.yml matches elsewhere are unrelated example fixtures and test files. The change is a clean, low-risk deletion with clear process tracking.

Code Review Details

No findings. The deletion is complete, the deprecation is intentional and tracked through the AppSec VULN-649 task, and no other files reference the removed workflow.

@theMickster theMickster marked this pull request as ready for review July 1, 2026 11:10
@theMickster theMickster requested a review from a team as a code owner July 1, 2026 11:10
@theMickster theMickster requested a review from fntyler July 1, 2026 11:10
@theMickster theMickster merged commit b8c5208 into main Jul 1, 2026
31 of 36 checks passed
@theMickster theMickster deleted the vuln-649/remove-scan-workflow branch July 1, 2026 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review version:skip

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants