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

Feb Snyk Review #7297

Merged
merged 2 commits into from
Feb 21, 2024
Merged

Feb Snyk Review #7297

merged 2 commits into from
Feb 21, 2024

Conversation

alismx
Copy link
Collaborator

@alismx alismx commented Feb 15, 2024

DEVOPS PULL REQUEST

Related Issue

Changes Proposed

  • Update packages in build.gradle (backend).

Testing

  • A smoke test of basic functionality should be accomplished before merging.
  • This PR has already been verified in pentest.

@alismx alismx changed the title chore: update project dependencies and their versions Feb Snyk Review Feb 15, 2024
@alismx alismx force-pushed the alis/7297-pins branch 3 times, most recently from f2d9f55 to cc40b9f Compare February 15, 2024 17:30

Verified

This commit was signed with the committer’s verified signature.
marekful Marcell Fülöp
@alismx alismx force-pushed the alis/7297-pins branch 5 times, most recently from 8fb733f to e104d3c Compare February 15, 2024 18:42
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@alismx alismx requested review from a team, mehansen, mpbrown and fzhao99 and removed request for a team February 20, 2024 18:25
@alismx alismx marked this pull request as ready for review February 20, 2024 18:25
Copy link
Contributor

@fzhao99 fzhao99 left a comment

Choose a reason for hiding this comment

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

tested on pentest and lgtm

Copy link
Collaborator

@mpbrown mpbrown left a comment

Choose a reason for hiding this comment

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

pentest lgtm!

@alismx alismx added this pull request to the merge queue Feb 21, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 21, 2024
@alismx alismx added this pull request to the merge queue Feb 21, 2024
Merged via the queue into main with commit 6233187 Feb 21, 2024
42 checks passed
@alismx alismx deleted the alis/7297-pins branch February 21, 2024 18:29
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.

None yet

3 participants