search

snyk / snyk-nuget-plugin

Basic Snyk CLI plugin for .NET support.
Other
5 stars 14 forks source link

feat: [OSM-347] Return multiple results part 2: Removing stuff we never used #173

Closed dotkas closed 11 months ago

dotkas commented 12 months ago

After https://github.com/snyk/snyk-nuget-plugin/pull/172 we want to remove the optional useRuntimeDependencies which was never used anywhere. We were already branching out if the option --dotnet-runtime-resolution was specified or not, so it did not make much sense. We already have a guard in that.

We've tested this for a while and it looks stable. It was inserted as a failsafe being over-cautious, but I feel comfortable removing it again now.

This should pave the road to enable the actual implementation of what we want, the multi-result solution for multiple <TargetFramework>s.

snyksec commented 11 months ago

:tada: This PR is included in version 1.35.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: