Skip to content

Conversation

@KeranYang
Copy link
Member

@KeranYang KeranYang commented May 20, 2025

Also fix CVEs.

Test: mvn clean install passed.

Signed-off-by: Keran Yang <yangkr920208@gmail.com>
@KeranYang KeranYang changed the title chore: upgrade some dependencies to their latest version (WIP)chore: upgrade some dependencies to their latest version May 20, 2025
.
Signed-off-by: Keran Yang <yangkr920208@gmail.com>
@codecov
Copy link

codecov bot commented May 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@2aeaaff). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #188   +/-   ##
=======================================
  Coverage        ?   58.76%           
  Complexity      ?      476           
=======================================
  Files           ?      151           
  Lines           ?     3390           
  Branches        ?      231           
=======================================
  Hits            ?     1992           
  Misses          ?     1231           
  Partials        ?      167           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@KeranYang KeranYang changed the title (WIP)chore: upgrade some dependencies to their latest version chore: upgrade some dependencies to their latest version May 20, 2025
@KeranYang KeranYang requested a review from yhl25 May 20, 2025 02:07
@KeranYang KeranYang marked this pull request as ready for review May 20, 2025 02:07
@KeranYang KeranYang requested review from Krithika3 and RohanAshar May 20, 2025 02:07
Copy link

@Krithika3 Krithika3 left a comment

Choose a reason for hiding this comment

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

Just one comment.

Copy link
Contributor

@RohanAshar RohanAshar left a comment

Choose a reason for hiding this comment

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

LGTM
Same comment as Krithika

@KeranYang
Copy link
Member Author

Folks had concerns about backward compatibility. Discussed offline, we will rely on unit tests + numaflow main e2e tests to verify.

@KeranYang KeranYang merged commit 25fe5e2 into main May 20, 2025
5 checks passed
@KeranYang KeranYang deleted the fix-cve branch May 20, 2025 18:47
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