-
Notifications
You must be signed in to change notification settings - Fork 6
chore: Springboot4 #419
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: refactor/refactoring_magma_traditional_in_new_magma
Are you sure you want to change the base?
chore: Springboot4 #419
Conversation
FBibonne
left a comment
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.
Remettre en cohérence les versions des dépendances utilisées avec celles fournies par Spring Boot 4
pom.xml
Outdated
| <groupId>com.fasterxml.jackson.jakarta.rs</groupId> | ||
| <groupId>tools.jackson.jakarta.rs</groupId> | ||
| <artifactId>jackson-jakarta-rs-json-provider</artifactId> | ||
| <version>2.17.2</version> |
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.
Cette dépendance n'existe pas : tools.jackson... c'est forcément du 3.0
| <jersey.version>4.0.0-M1</jersey.version> | ||
| <jackson.version>2.17.2</jackson.version> | ||
| <jersey.version>4.0.0</jersey.version> | ||
| <jackson.version>3.0.0</jackson.version> |
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.
les versions de jackson sont gérées par spring boot : inutile de surcharger
| <!-- Bind csv into POJOs --> | ||
| <dependency> | ||
| <groupId>com.fasterxml.jackson.dataformat</groupId> | ||
| <groupId>tools.jackson.dataformat</groupId> |
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.
les versions de jackson sont gérées par spring boot : inutile de surcharger
| <!-- XML mapping --> | ||
| <dependency> | ||
| <groupId>com.fasterxml.jackson.dataformat</groupId> | ||
| <groupId>tools.jackson.dataformat</groupId> |
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.
les versions de jackson sont gérées par spring boot : inutile de surcharger
pom.xml
Outdated
| <swagger.version>2.2.22</swagger.version> | ||
| <mockito.version>5.12.0</mockito.version> | ||
| <swagger.version>2.2.41</swagger.version> | ||
| <mockito.version>5.21.0</mockito.version> |
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.
les versions de mockito sont gérées par spring boot
pom.xml
Outdated
| <mockito.version>5.21.0</mockito.version> | ||
| <freemarker.version>2.3.34</freemarker.version> | ||
| <xmlunit.version>2.10.0</xmlunit.version> | ||
| <xmlunit.version>2.11.0</xmlunit.version> |
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.
les versions de xmlunit sont gérées par spring boot
| ObjectReader reader = csvMapper.readerFor(targetClass).with(schema); | ||
| List<G> results; | ||
| try (MappingIterator<G> mappingIterator = reader.readValues(csv)) { | ||
| results = mappingIterator.readAll(); |
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.
Est on certain avec la suppression du cast qu'on veut retourner une erreur lorsqu'il y a un pb de désérialisation ?
|
Pas de choses supplementaires que celles de Fabrice |
xmlunit and mockito versions are managed by spring boot tools.jackson.jakarta.rs dependency does not exist
No description provided.