Skip to content

Conversation

@parker-snyk
Copy link
Contributor

There were a few areas for improvement in the code base.

These changes were pulled from #690 as they were out of scope

@parker-snyk parker-snyk marked this pull request as ready for review August 11, 2025 18:37
@parker-snyk parker-snyk requested a review from a team as a code owner August 11, 2025 18:37
@parker-snyk parker-snyk requested a review from bdemeo12 August 11, 2025 18:37
cursor[bot]

This comment was marked as outdated.

@bdemeo12
Copy link
Contributor

Based on your commit here: 1e080f2 these changes look good. Your tests are failing, and when I pull your branch and run them, they fail there for me too, but seem to be failing on main as well: https://app.circleci.com/pipelines/github/snyk/snyk-docker-plugin?branch=main.

@parker-snyk parker-snyk enabled auto-merge (rebase) August 12, 2025 20:55
@parker-snyk parker-snyk force-pushed the chore/CN-276-cleanup branch from bbc1b5b to 85220d3 Compare August 12, 2025 21:06
auto-merge was automatically disabled August 12, 2025 21:29

Base branch requires signed commits

@parker-snyk parker-snyk merged commit e6410b5 into main Aug 12, 2025
13 checks passed
@parker-snyk parker-snyk deleted the chore/CN-276-cleanup branch August 12, 2025 21:30
@bdemeo12
Copy link
Contributor

You're the bomb for fixing those tests 🚢 !

@snyksec
Copy link

snyksec commented Aug 12, 2025

🎉 This PR is included in version 8.5.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants