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

CORE-15403: Update Snyk vulnerability #4316

Merged
merged 3 commits into from
Jul 18, 2023
Merged

Conversation

nargas-ritu
Copy link
Contributor

CORE - 15403: Update Snyk vulnerability

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR title failed to match regex -> ^((CORDA|EG|ENT|INFRA|CORE|DOC|ES)-\d+)(.*)

@vkolomeyko vkolomeyko changed the title CORE - 15403: Update Snyk vulnerability CORE-15403: Update Snyk vulnerability Jul 17, 2023
vkolomeyko
vkolomeyko previously approved these changes Jul 17, 2023
Copy link
Contributor

@vkolomeyko vkolomeyko left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot dismissed their stale review July 17, 2023 13:21

All good!

yarivus
yarivus previously approved these changes Jul 17, 2023
Copy link
Contributor

@yarivus yarivus left a comment

Choose a reason for hiding this comment

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

LGTM

@corda-jenkins-ci02
Copy link
Contributor

corda-jenkins-ci02 bot commented Jul 17, 2023

Jenkins build for PR 4316 build 3

Build Successful:
Jar artifact version produced by this PR: 5.0.0.0-alpha-1689672967962
Helm chart version produced by this PR: 5.0.0-alpha.1689672967962
Helm chart pushed to: corda-os-docker-dev.software.r3.com/helm-charts/pr-4316/corda

@nargas-ritu nargas-ritu dismissed stale reviews from yarivus and vkolomeyko via e909865 July 18, 2023 09:26
.snyk Outdated Show resolved Hide resolved
Co-authored-by: Yariv Bachar <104990295+yarivus@users.noreply.github.com>
Copy link
Contributor

@vkolomeyko vkolomeyko left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@yarivus yarivus left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ronanbrowne ronanbrowne left a comment

Choose a reason for hiding this comment

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

LGTM

@nargas-ritu nargas-ritu merged commit 1116f52 into release/os/5.0 Jul 18, 2023
@nargas-ritu nargas-ritu deleted the nargas-ritu-patch-1 branch July 18, 2023 10:28
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.

4 participants