Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added the call for RunAnalysis to Trigger Analysis API #30

Merged
merged 6 commits into from
Apr 23, 2024

Conversation

acke
Copy link
Contributor

@acke acke commented Apr 16, 2024

This commit integrates the new Analysis API in Snyk Code scanning flow.

It does poll for result, but does not actually reads out the findings.

@CLAassistant
Copy link

CLAassistant commented Apr 16, 2024

CLA assistant check
All committers have signed the CLA.

@acke acke force-pushed the feat/IDE-194-integrate-new-analysis-api-in-scanning-flow branch 5 times, most recently from 646ab02 to cb84671 Compare April 19, 2024 11:44
@acke acke marked this pull request as ready for review April 19, 2024 15:52
@acke acke requested a review from a team as a code owner April 19, 2024 15:52
@acke
Copy link
Contributor Author

acke commented Apr 19, 2024

This commit integrates the new Analysis API in Snyk Code scanning flow.

It does poll for result, but does not actually reads out the findings.

@acke acke marked this pull request as draft April 19, 2024 15:58
@acke acke marked this pull request as ready for review April 22, 2024 07:32
@acke acke marked this pull request as draft April 22, 2024 07:33
@acke acke force-pushed the feat/IDE-194-integrate-new-analysis-api-in-scanning-flow branch from eadf543 to 2132d3a Compare April 22, 2024 09:20
co-authored-by: Bastian Doetsch <bastian.doetsch@snyk.io>
co-authored-by: Teodora Sandu <teodora.sandu@snyk.io>
@acke acke force-pushed the feat/IDE-194-integrate-new-analysis-api-in-scanning-flow branch from 2132d3a to f5f1b18 Compare April 22, 2024 09:26
@acke acke marked this pull request as ready for review April 22, 2024 09:27
acke and others added 4 commits April 22, 2024 12:20
Co-authored-by: Teodora Sandu <81559517+teodora-sandu@users.noreply.github.com>
Signed-off-by: Knut Funkel <knut.funkel@snyk.io>
Co-authored-by: Teodora Sandu <81559517+teodora-sandu@users.noreply.github.com>
Signed-off-by: Knut Funkel <knut.funkel@snyk.io>
Co-authored-by: Teodora Sandu <81559517+teodora-sandu@users.noreply.github.com>
Signed-off-by: Knut Funkel <knut.funkel@snyk.io>
@acke acke force-pushed the feat/IDE-194-integrate-new-analysis-api-in-scanning-flow branch from 22eca46 to 598482a Compare April 22, 2024 15:13
@acke acke merged commit 4efdd02 into main Apr 23, 2024
9 checks passed
@acke acke deleted the feat/IDE-194-integrate-new-analysis-api-in-scanning-flow branch April 23, 2024 08:16
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.

3 participants