-
Notifications
You must be signed in to change notification settings - Fork 320
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
fix: Signe code to do the PR #2563
fix: Signe code to do the PR #2563
Conversation
This branch is a clone of renovate/web use for the PR MarquezProject#2520.
Codecov Report
@@ Coverage Diff @@
## main #2563 +/- ##
=========================================
Coverage 83.26% 83.26%
Complexity 1286 1286
=========================================
Files 243 243
Lines 5934 5934
Branches 279 279
=========================================
Hits 4941 4941
Misses 846 846
Partials 147 147 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Xavier-Cliquennois This is tight! Thanks for all the sweet upgrades! @phixMe since you approved #2520, I'm assuming we're good to merge.
Great job! Congrats on your first merged pull request in the Marquez project! |
Problem
👋 Thanks for opening a pull request! Please include a brief summary of the problem your change is trying to solve, or bug fix. If your change fixes a bug or you'd like to provide context on why you're making the change, please link the issue as follows:
Closes: None
Solution
Please describe your change as it relates to the problem, or bug fix, as well as any dependencies. If your change requires a database schema migration, please describe the schema modification(s) and whether it's a backwards-incompatible or backwards-compatible change.
One-line summary: The goal was to upgrade the web client to React 18 in order to utilize the latest version of Node.js and update all dependencies to their respective latest versions. Everything is good only the unit tests haven't been fully migrated to Jest and test-library for the moment.
This PR is a clone of the PR 2520. It just a signed version.
Checklist
CHANGELOG.md
(Depending on the change, this may not be necessary)..sql
database schema migration according to Flyway's naming convention (if relevant)