Skip to content

Strict checks on null values for models directory and its dependencies#708

Draft
Pivouane wants to merge 6 commits intomainfrom
psa/strict-null-check-models
Draft

Strict checks on null values for models directory and its dependencies#708
Pivouane wants to merge 6 commits intomainfrom
psa/strict-null-check-models

Conversation

@Pivouane
Copy link
Copy Markdown
Contributor

Description

Handle null values warnings and errors for the models directory and its dependencies

Issues

#645

Checklist

  • This PR contains a description of the changes I'm making
  • I've read the Contribution Guidelines
  • I've added tests for changes or features I've introduced
  • I documented any high-level concepts I'm introducing in documentation/
  • CI is currently green and this is ready for review

Pivouane and others added 6 commits December 22, 2025 11:57
Signed-off-by: Pivouane (Pierre Santamaria) <pierrenumber1@outlook.com>
Signed-off-by: pierre santamaria <pierre.santamaria@epita.fr>
Signed-off-by: Pivouane (Pierre Santamaria) <pierrenumber1@outlook.com>
… directory app/data-structures

Signed-off-by: Pivouane (Pierre Santamaria) <pierrenumber1@outlook.com>
…ctory app/services

Signed-off-by: Pivouane (Pierre Santamaria) <pierrenumber1@outlook.com>
Signed-off-by: Pivouane (Pierre Santamaria) <pierrenumber1@outlook.com>
@Pivouane Pivouane linked an issue Dec 22, 2025 that may be closed by this pull request
7 tasks
@aiAdrian
Copy link
Copy Markdown
Contributor

I am not sure whether we have to migrate the new added locked to the data migration. Netzgrafik-Editor must ensure that elder version will get automatically updated to latest version !

Please dbl-check if required to migrate - if yes -> add it to :

static migrateNetzgrafikDto(netzgrafikDto: NetzgrafikDto) {

https://github.com/OpenRailAssociation/netzgrafik-editor-frontend/blob/main/src/app/utils/data-migration.ts#L50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ci GitHub actions code-quality Technical enhancements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature request]: Add StrictNullCheck constraint to configuration

3 participants