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

chore: align dependency versions with flow/platform 24.4 #2002

Merged
merged 6 commits into from
Feb 1, 2024

Conversation

ZheSun88
Copy link
Contributor

align with platform 24.4

align with platform 24.4
Copy link

codecov bot commented Jan 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c09ced9) 95.30% compared to head (7ed759f) 95.30%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2002   +/-   ##
=======================================
  Coverage   95.30%   95.30%           
=======================================
  Files          52       52           
  Lines        3495     3495           
  Branches      527      527           
=======================================
  Hits         3331     3331           
  Misses        129      129           
  Partials       35       35           
Flag Coverage Δ
unittests 95.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@ZheSun88 ZheSun88 enabled auto-merge (squash) January 30, 2024 15:38
@ZheSun88 ZheSun88 changed the title chore: upgrade jna version to 5.14.0 chore: align dependency versions with flow/platform 24.4 Jan 31, 2024
pom.xml Outdated
<swagger.core.version>2.2.20</swagger.core.version>
<swagger.models.version>2.2.20</swagger.models.version>
<swagger.parser.v3.version>2.1.15</swagger.parser.v3.version>
<jackson.version>2.15.3</jackson.version>
<jackson.version>2.16.0</jackson.version>
Copy link
Contributor

Choose a reason for hiding this comment

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

We have Snyk PRs asking for 2.16.1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

flow has reverted the 2.16.1 vaadin/flow@e6028dd

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there is an issue about the enforcer-plugin. but i dont quite understand why it fail in this way.

Dependency convergence error for com.fasterxml.jackson.datatype:jackson-datatype-jdk8:jar:2.15.3 paths to dependency are:
Error:  +-com.vaadin.hilla:parser-jvm-plugin-backbone:jar:24.4-SNAPSHOT
Error:    +-com.vaadin.hilla:parser-jvm-utils:jar:24.4-SNAPSHOT:compile
Error:      +-com.fasterxml.jackson.datatype:jackson-datatype-jdk8:jar:2.15.3:compile
Error:  and
Error:  +-com.vaadin.hilla:parser-jvm-plugin-backbone:jar:24.4-SNAPSHOT
Error:    +-com.fasterxml.jackson.datatype:jackson-datatype-jdk8:jar:2.16.0:compile

the parser-jvm-utils is a module we build and version used there is from the root of hilla project. where is the 2.15.3 coming.. aww.

Copy link
Contributor

Choose a reason for hiding this comment

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

Let me have a look

Copy link
Contributor

Choose a reason for hiding this comment

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

It comes from Swagger. I'll try to align all dependencies.

Copy link
Contributor

Choose a reason for hiding this comment

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

Well, Swagger is up to date, so I guess we're not ready to use Jackson 2.16 in Hilla since Swagger itself still doesn't use it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

okay.. then let us just revert the jackson version update.

Copy link

sonarcloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ZheSun88 ZheSun88 merged commit a28d45a into main Feb 1, 2024
15 checks passed
@ZheSun88 ZheSun88 deleted the ZheSun88-patch-2 branch February 1, 2024 13:11
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.

3 participants