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

WIP: try reporting results to snyk.io #1248

Closed
wants to merge 2 commits into from

Conversation

chmeliik
Copy link
Contributor

@chmeliik chmeliik commented Aug 5, 2024

Before you complete this pull request ...

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

@chmeliik
Copy link
Contributor Author

chmeliik commented Aug 5, 2024

/retest

Signed-off-by: Adam Cmiel <[email protected]>
@chmeliik
Copy link
Contributor Author

chmeliik commented Aug 6, 2024

/retest

1 similar comment
@chmeliik
Copy link
Contributor Author

chmeliik commented Aug 6, 2024

/retest

@chmeliik
Copy link
Contributor Author

chmeliik commented Aug 6, 2024

Yup, uploading the reports on PRs definitely does cause the PRs to overwrite each other's results

When #1252 uploads its report:

image

When this PR uploads its report:

image

@chmeliik chmeliik closed this Aug 6, 2024
@chmeliik chmeliik reopened this Aug 6, 2024
@chmeliik chmeliik closed this Aug 6, 2024
@chmeliik chmeliik deleted the snyk-report-wip branch August 6, 2024 11:46
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.

1 participant