-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Update packages #7268
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
base: master
Are you sure you want to change the base?
Update packages #7268
Conversation
This Pull Request has failed the formatting check Please run You can automate this auto-formatting process to execute on the git pre-push hook, by installing pre-commit and then calling |
Hi @madduci - Thanks for the contribution! This looks to be failing quite a few CI tests though, it's not currently in a state that could be merged. |
Hi @jamesagnew yes after the upgrade of packages, some tests are failing, currently i don't know if the expected wrong actual values in tests are there because of the internal processing of the tinder-plugin, which seems to behave differently after the upgrade of some plugins/libraries |
31928b3
to
d1d4e5a
Compare
I'm happy to weigh in on specific issues, but there are just so many CI failures in this PR. It might be worth breaking this into smaller changes with fewer dependency upgrades so that we could discuss what is breaking. |
Hi
I've performed an upgrade of many libraries to resolve important CVEs.
I had to change some code to address deprecated / removed / unused code.
Current Status: some tests are still failing, mostly due to dirty context, i am trying to correct them
I've also applied
mvn spotless:apply
, but the GitHub Action is still failing