Conversation
|
I'll organize the PR in several commits when it's building. |
a069a72 to
0ff863f
Compare
5474553 to
d3ebf77
Compare
e99017c to
68cf7d3
Compare
|
Here the most impacting changes :
Feel free to squash the commits into one |
99fda23 to
91f6a5b
Compare
91f6a5b to
8d17e5f
Compare
pcdavid
left a comment
There was a problem hiding this comment.
Note that 4.0.5 has just been released: https://spring.io/blog/2026/03/26/spring-boot-4-0-5-available-now
|
|
||
| === Dependency update | ||
|
|
||
| - [releng] Switch to Spring Boot 4.0 |
There was a problem hiding this comment.
Mention the exact version number.
| <groupId>org.springframework.boot</groupId> | ||
| <artifactId>spring-boot-starter-liquibase</artifactId> |
There was a problem hiding this comment.
We can't upgrade to liquibase 5.x
| package org.eclipse.sirius.web.papaya.representations.table; | ||
|
|
||
| import com.fasterxml.jackson.databind.ObjectMapper; | ||
| import tools.jackson.databind.ObjectMapper; |
There was a problem hiding this comment.
Imports need to be re-ordered
Pull request template
General purpose
What is the main goal of this pull request?
Project management
priority:andpr:labels been added to the pull request? (In case of doubt, start with the labelspriority: lowandpr: to review later)area:,difficulty:,type:)CHANGELOG.adocbeen updated to reference the relevant issues?CHANGELOG.adoc? (Including changes in the GraphQL API)CHANGELOG.adoc? For example indoc/screenshots/2022.5.0-my-new-feature.pngArchitectural decision records (ADR)
[doc]?CHANGELOG.adoc?Dependencies
CHANGELOG.adoc?CHANGELOG.adoc?Frontend
This section is not relevant if your contribution does not come with changes to the frontend.
General purpose
Typing
We need to improve the typing of our code, as such, we require every contribution to come with proper TypeScript typing for both changes contributing new files and those modifying existing files.
Please ensure that the following statements are true for each file created or modified (this may require you to improve code outside of your contribution).
useMutation<DATA_TYPE, VARIABLE_TYPE>(…)useQuery<DATA_TYPE, VARIABLE_TYPE>(…)useSubscription<DATA_TYPE, VARIABLE_TYPE>(…)useMachine<CONTEXT_TYPE, EVENTS_TYPE>(…)useState<STATE_TYPE>(…)?.(if the GraphQL API specifies that a field cannot benull, do not treat it has potentiallynullfor example)let diagram: Diagram | null = null;)Backend
This section is not relevant if your contribution does not come with changes to the backend.
General purpose
Architecture
Review
How to test this PR?
Please describe here the various use cases to test this pull request