Skip to content

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

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

[VULN-649] ci: Remove deprecated scan workflow#149
theMickster merged 2 commits 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 deletes the deprecated .github/workflows/scan.yml GitHub Actions workflow (the Checkmarx SAST and check-run jobs), tracked under VULN-649. The removal is complete — no remaining workflow, script, or config references the deleted scan.yml or its check-run/sast jobs. The remaining scan.yml/Checkmarx mentions in the bitwarden-security-engineer plugin describe the generic Bitwarden security toolchain present in every repository and are not a dependency on this repo's deleted file. As a CI-infrastructure-only change with no plugin code, docs, or config touched, no plugin version bump or changelog entry is required.

Code Review Details

No findings. The deletion is clean and well-scoped.

@theMickster theMickster marked this pull request as ready for review July 1, 2026 10:58
@theMickster theMickster requested a review from a team as a code owner July 1, 2026 10:58
@theMickster theMickster enabled auto-merge (squash) July 1, 2026 13:32
@theMickster theMickster merged commit a6af8cf into main Jul 1, 2026
9 checks passed
@theMickster theMickster deleted the vuln-649/remove-scan-workflow branch July 1, 2026 13:35
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants