Skip to content

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

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

[VULN-649] ci: Remove deprecated scan workflow#504
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 removes the deprecated scan.yml GitHub Actions workflow, which previously ran Checkmarx SAST scanning. The change is tracked under VULN-649, indicating AppSec ownership of the deprecation decision. I verified the deletion is complete — no other workflows, scripts, or files in the repository reference scan.yml, the Checkmarx workflow, the check-run job, or the Scan workflow name. No issues found.

Code Review Details

No actionable findings.

@theMickster theMickster marked this pull request as ready for review July 1, 2026 10:59
@theMickster theMickster requested a review from a team as a code owner July 1, 2026 10:59
@theMickster theMickster requested a review from audreyality July 1, 2026 10:59

@audreyality audreyality left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

📝 BIT testing has become unstable recently. That's not material to this change, but I merged up from main (where they're fixed) for good measure.

@theMickster theMickster merged commit 9ea1e1f into main Jul 1, 2026
15 checks passed
@theMickster theMickster deleted the vuln-649/remove-scan-workflow branch July 1, 2026 13:09
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.

4 participants