Skip to content

ci: remove Snyk scan and monitor - #1142

Merged
brunodam merged 2 commits into
mainfrom
1141-remove-snyk
Nov 6, 2026
Merged

brunodam merged 2 commits into
mainfrom
1141-remove-snyk

Conversation

@brunodam

Copy link
Copy Markdown
Contributor

Description

Snyk is being decommissioned across the hiero-ledger, hashgraph, and swirldslabs organizations. This removes it from CI here.

This pull request changes the following:

  • Delete zxf-snyk-monitor.yaml
  • Remove the Snyk Scan job from flow-pull-request-checks.yaml
  • Remove the Snyk steps, the enable-snyk-scan input, and the snyk-token secret from zxc-code-analysis.yaml; Sonar and CodeCov analysis are unchanged
  • Stop passing enable-snyk-scan and SNYK_TOKEN from flow-build-application.yaml, and drop its enable-snyk-scan dispatch input

Edited workflows parse as YAML, and actionlint reports no new findings on them. No ruleset requires a Snyk check in this repository.

Related Issues

Signed-off-by: Bruno De Assis Marques <bruno.marques@swirldslabs.com>
Copilot AI balanced review requested due to automatic review settings October 30, 2026 05:17
@brunodam brunodam self-assigned this Oct 30, 2026
@brunodam
brunodam requested review from a team as code owners October 30, 2026 05:17
@codacy-production

codacy-production Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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.

🟢 Approval recommended

Repository-wide searches confirm every Snyk job, input, secret, and caller reference was fully removed with no orphaned references and all edited workflows parse as valid YAML.

No open issues 🎉

What changed in this PR

This PR removes Snyk from CI as part of its decommissioning across the hiero-ledger, hashgraph, and swirldslabs organizations (closes #1141). It deletes the standalone Snyk monitor workflow and strips all Snyk-related jobs, steps, inputs, and secrets from the reusable code-analysis workflow and its callers, leaving Sonar and CodeCov analysis intact.

Changes:

  • Delete the ZXF: Snyk Monitor workflow entirely.
  • Remove the Snyk Scan job from PR checks and the Snyk steps/enable-snyk-scan input/snyk-token secret from the reusable code-analysis workflow.
  • Stop passing enable-snyk-scan and SNYK_TOKEN from the build workflow and drop its enable-snyk-scan dispatch input.
File Description
.github/​workflows/​zxf-snyk-monitor.yaml Deleted; removes the scheduled/push-triggered Snyk monitor job.
.github/​workflows/​zxc-code-analysis.yaml Removes the enable-snyk-scan input, snyk-token secret, all Snyk steps, and updates the Node setup comment.
.github/​workflows/​flow-pull-request-checks.yaml Removes the Snyk Scan job that invoked the analysis workflow with Snyk enabled.
.github/​workflows/​flow-build-application.yaml Drops the enable-snyk-scan dispatch input and stops passing it plus SNYK_TOKEN to the analysis workflow.

Review effort: Balanced

Co-authored-by: brunodam <83957476+brunodam@users.noreply.github.com>
@brunodam
brunodam merged commit 50263fd into main Nov 6, 2026
14 of 17 checks passed
@brunodam
brunodam deleted the 1141-remove-snyk branch November 6, 2026 02:25
Copilot stopped work on behalf of brunodam due to an error November 6, 2026 02:26
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.

Remove Snyk from CI

5 participants