Skip to content

Conversation

@saveliy-sviridov
Copy link
Contributor

No description provided.

ssviridov and others added 17 commits October 31, 2024 15:13
…om deserializer and add other.model property to edxl de
…, modify json2xsd code to use a yaml containing schema data on input instead of relying on a hard-coded array, add templates necessary to generate generator config files and ContentMessage class, add placeholder schemas.yaml files
…us schema properties used in automatic file generation, update mdd manually to temporarily make the automatic generation script work
…/mdd/automatic-schema-generation

# Conflicts:
#	csv_parser/models/model-technical.xlsx
#	csv_parser/models/model.xlsx
@saveliy-sviridov saveliy-sviridov added the WIP Work In Progress: the branch is still under active development and not stable / in a working state label Nov 28, 2024
Copy link
Contributor

@romainfd romainfd left a comment

Choose a reason for hiding this comment

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

All good, on va pouvoir merger ça !! Peut-être rajouter les commentaires en haut des fichiers et sortir les licences des templates ?

Copy link
Contributor

@romainfd romainfd left a comment

Choose a reason for hiding this comment

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

De mieux en mieux !! On y est presque 🗡️

@romainfd
Copy link
Contributor

Je me demande s'il va pas falloir rajouter une étape qui nettoie config/generated/ avant de regénérer ? Et enlever à la main config/auto/ aussi du coup
Et idem pour les autres endroits où on génère des choses, si on change un nom de schéma ou qu'on l'enlève, ça supprimer bien tous les outputs associés ?

@sonarqubecloud
Copy link

Copy link
Contributor

@romainfd romainfd left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!!!!

@romainfd romainfd merged commit 864d1e0 into auto/model_tracker Dec 16, 2024
2 checks passed
@romainfd romainfd deleted the feature/mdd/automatic-schema-generation branch December 16, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review Ready The branch is ready for review (in a working state and/or review required)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants