etalab / schema-irve

TableSchema pour les Infrastructures de Recharge de Véhicules Electriques (IRVE)
10 stars 10 forks source link

Ajout du préfixe "v" devant le numéro de version + intégration de "checks" automatisés #58

Closed thbar closed 5 months ago

thbar commented 5 months ago

Je réalise ici un test sur ce qu'a implémenté @Pierlou :

EDIT: le test est concluant, j'ai corrigé le numéro de version, et on intègre ça pour la suite.

Voir:

thbar commented 5 months ago

@Pierlou j'ai ajouté "v" devant la version, est-ce que tu confirmes que cela fonctionnera bien sur les différents outils actuellement (schema.data.gouv.fr, validata etc) ?

thbar commented 5 months ago

@etalab/transport-tech review davantage pour partager la connaissance là dessus que pour un vrai besoin de validation à ce stade !

AntoineAugusti commented 5 months ago

Ah je fais une review ici alors que ça concerne plus l'autre dépôt, ça m'apprendra à lire la description. cc @Pierlou

AntoineAugusti commented 5 months ago

J'aurais dû avoir la puce à l'oreille en voyant du Python 🐍 dans une PR de Thibaut 😏

thbar commented 5 months ago

Ah je fais une review ici alors que ça concerne plus l'autre dépôt, ça m'apprendra à lire la description. cc @Pierlou

Aucun souci, c'est en fait une bonne remarque dans le sens où @Pierlou on gagnerait à avoir une "GitHub action publiée" qu'on référence (comme le classique https://github.com/actions/checkout) dans chaque repo à terme, pour éviter le copier-collé, une fois que ça sera rodé !

J'aurais dû avoir la puce à l'oreille en voyant du Python 🐍 dans une PR de Thibaut 😏

Hihi :-)

Pierlou commented 5 months ago

j'ai ajouté "v" devant la version, est-ce que tu confirmes que cela fonctionnera bien sur les différents outils actuellement (schema.data.gouv.fr, validata etc) ?

Oui ça se passe bien, c'est le cas notamment dans ce schéma

thbar commented 5 months ago

J'ai backporté les derniers changements apportés à https://github.com/datagouv/datapackage-template/pull/1.

thbar commented 5 months ago

@AntoineAugusti @Pierlou à vot' bon coeur pour une revue finale (qui n'atterrira pas à ce stade dans une nouvelle version je précise).