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

[importer] Permettre l'import de toutes les règles enfant #42

Open
Clemog opened this issue May 22, 2024 · 2 comments
Open

[importer] Permettre l'import de toutes les règles enfant #42

Clemog opened this issue May 22, 2024 · 2 comments
Labels
✨ enhancement New feature or request

Comments

@Clemog
Copy link
Contributor

Clemog commented May 22, 2024

Lors de l'import de règles depuis un autre package, ce serait intéressant d'automatiser l'import de toutes les règles enfant d'un espace nom donné

@Clemog Clemog added the ✨ enhancement New feature or request label May 22, 2024
@Clemog Clemog changed the title Permettre l'import de toutes les règles enfant [importer] Permettre l'import de toutes les règles enfant May 22, 2024
@Clemog
Copy link
Contributor Author

Clemog commented May 22, 2024

Exemple :

Eviter de genre d'imports auxquels vont s'ajouter ceux pour l'eau

importer!:
  depuis:
    nom: '@incubateur-ademe/publicodes-negaoctet'
    url: https://github.com/incubateur-ademe/publicodes-negaoctet
  dans: negaoctet
  les règles:
    - acv . smartphone . carbone . fabrication
    - acv . smartphone . carbone . distribution
    - acv . smartphone . durée de vie théorique
    - acv . tablette . carbone . fabrication
    - acv . tablette . carbone . distribution
    - acv . tablette . durée de vie théorique
    - acv . ordinateur portable . carbone . fabrication
    - acv . ordinateur portable . carbone . distribution
    - acv . ordinateur portable . durée de vie théorique
    - acv . TV . carbone . fabrication
    - acv . TV . carbone . distribution
    - acv . TV . durée de vie théorique
    - acv . ordinateur fixe . carbone . fabrication
    - acv . ordinateur fixe . carbone . distribution
    - acv . ordinateur fixe . durée de vie théorique
    - acv . écran . carbone . fabrication
    - acv . écran . carbone . distribution
    - acv . écran . durée de vie théorique
    - acv . enceinte . carbone . fabrication
    - acv . enceinte . carbone . distribution
    - acv . enceinte . durée de vie théorique
    - acv . console de salon . carbone . fabrication
    - acv . console de salon . carbone . distribution
    - acv . console de salon . durée de vie théorique
    - acv . console portable . carbone . fabrication
    - acv . console portable . carbone . distribution
    - acv . console portable . durée de vie théorique
    - acv . imprimante . carbone . fabrication
    - acv . imprimante . carbone . distribution
    - acv . imprimante . durée de vie théorique

@EmileRolley
Copy link
Collaborator

Je ne suis pas sûr que ce soit une si bonne idée. En effet, il faudrait inventer une nouvelle syntaxe pour spécifier l'import ou non de toutes les règles enfants et donc qui complexifierait la lecture et la compréhension du code. L'avantage actuellement est que l'on sait explicitement toutes les règles qui seront importées.

A moins d'avoir vraiment une bonne raison de le faire je suis pour ne pas ajouter cette fonctionnalité pour l'instant.

Note

On pourrait imaginer avoir un attribut les espaces-noms en plus de les règles mais il faudrait que la personne qui lit le code comprenne la différence entre un espace-nom et une règle (ce qui n'est pas très clair en soit de base dans Publicodes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
Status: Long term todo
Development

No branches or pull requests

2 participants