Skip to content

Feat: show confidence score - #2363

Merged
danlucian merged 1 commit into
masterfrom
feat/show-confidence-score
Nov 19, 2021
Merged

Feat: show confidence score#2363
danlucian merged 1 commit into
masterfrom
feat/show-confidence-score

Conversation

@danlucian

@danlucian danlucian commented Nov 12, 2021

Copy link
Copy Markdown
Contributor

What does this PR do?

The purpose of those changes are related to the addition of a new field (called confidence) in the CLI output for the unmanaged flows.

When using either --print-deps or --print-dep-paths args along with the unmanaged scan command on the CLI you should be able to see the confidence score of the found dependencies.

How should this be manually tested?

Yes

What are the relevant tickets?

TUN-58

Screenshot

--print-deps
Screenshot 2021-11-12 at 18 28 31

or

--print-dep-paths
Screenshot 2021-11-12 at 18 29 35

@github-actions

github-actions Bot commented Nov 12, 2021

Copy link
Copy Markdown
Contributor
Warnings
⚠️

Since the CLI is unifying on a standard and improved tooling, we're starting to migrate old-style imports and exports to ES6 ones.
A file you've modified is using either module.exports or require(). If you can, please update them to ES6 import syntax and export syntax.
Files found:

  • src/lib/snyk-test/legacy.ts

Generated by 🚫 dangerJS against b21aab7

@ekbsnyk
ekbsnyk force-pushed the feat/show-confidence-score branch from 38753c7 to 3bf725d Compare November 12, 2021 16:58
Comment thread webpack.common.js
Comment thread check-dependencies.config.ts
@danlucian
danlucian force-pushed the feat/show-confidence-score branch 5 times, most recently from 8d5e8b4 to d62d1d9 Compare November 16, 2021 13:50

@anthogez anthogez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

missing new version from pr snyk/snyk-cpp-plugin#52

The purpose of those changes are related to the addition of a new field (called
confidence) in the CLI output for the unmanaged flows.

When using either --print-deps or --print-dep-paths args along with the
unmanaged scan command on the CLI you should be able to see the confidence score of the found dependencies.

Co-authored-by: daniel.ekelund@snyk.io
Co-authored-by: eleanor.kavanagh-brown@snyk.io
@danlucian
danlucian force-pushed the feat/show-confidence-score branch from d62d1d9 to b21aab7 Compare November 18, 2021 08:10
@danlucian

Copy link
Copy Markdown
Contributor Author

missing new version from pr snyk/snyk-cpp-plugin#52

Updated.

@dekelund
dekelund self-requested a review November 19, 2021 07:59
@danlucian
danlucian merged commit b84c068 into master Nov 19, 2021
@danlucian
danlucian deleted the feat/show-confidence-score branch November 19, 2021 08:12
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.

5 participants