Skip to content

[SC-81803] Remove Snyk policy file - #8

Merged
jcesarioatpeach merged 1 commit into
masterfrom
johncesario/sc-81803/remove-snyk-policy-file-from-peachfinance
Sep 29, 2026
Merged

jcesarioatpeach merged 1 commit into
masterfrom
johncesario/sc-81803/remove-snyk-policy-file-from-peachfinance

Conversation

@jcesarioatpeach

Copy link
Copy Markdown

Summary

  • Deletes the .snyk policy file. Snyk is being decommissioned org-wide (sc-81450, parent sc-80836) in favor of Wiz and Ox.
  • Snyk webhook removal is tracked separately in sc-80836 — not touched here.

AC verification

  1. .snyk is deleted. rg -i snyk in the repo still returns 2 hits, both coincidental base64 substrings inside a merchantLogo image blob in generator/cybersource-rest-spec-python.json and generator/cybersource-rest-spec.json — pre-existing test fixture data, not real Snyk references. No actual Snyk references remain.
  2. CI is expected to be green (deletion-only diff, no production code path touched).
  3. Org audit on 2026-09-29 confirmed no Snyk check is a required status check on this repo.

Fixes [sc-81803]

🤖 Generated with Claude Code

Snyk is being decommissioned org-wide (sc-81450); this repo's .snyk
policy file is no longer needed. Webhook removal is tracked separately
in sc-80836.

Fixes [sc-81803]

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The deletion is complete, scoped, and does not affect production code.

Review effort: Balanced
Findings: None

What changed in this PR

Removes the obsolete Snyk policy following the organization-wide Snyk decommissioning.

Changes:

  • Deletes the .snyk policy file.
  • Leaves no functional Snyk references in the repository.
File Description
.snyk Deletes obsolete Snyk exclusions and policy configuration.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jcesarioatpeach
jcesarioatpeach marked this pull request as ready for review September 29, 2026 17:52
@jcesarioatpeach
jcesarioatpeach merged commit ab08e56 into master Sep 29, 2026
2 checks passed
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