fix: remove isPublishable logic

snyk / snyk-nuget-plugin

Basic Snyk CLI plugin for .NET support.

Other

7 stars 14 forks source link

fix: remove isPublishable logic #212

Closed andrei-cacio closed 4 months ago

andrei-cacio commented 5 months ago

[ ] Ready for review
[ ] Follows CONTRIBUTING rules
[ ] Reviewed by Snyk internal team

What does this PR do?

the isPublishable was added to make better error messages for users who try to scan a project defined be not publishable.

However, this flag causes issues with older .NET versions. Will be removed until a better solution is in place.

CLAassistant commented 5 months ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

snyksec commented 4 months ago

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

The release is available on:

Your semantic-release bot :package::rocket: