etalab / schema-irve

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

Correctifs multiples #46

Closed thbar closed 8 months ago

thbar commented 8 months ago

Dans cette PR:

À l'issue, je publierai la v2.2.1, qui ne change rien au schéma lui-même (mais corrige donc la doc).

AntoineAugusti commented 8 months ago

Que penses-tu d'ajouter la contrainte de https://github.com/etalab/schema-irve/issues/36 ? Ou dans une version ultérieure si celle-ci gère uniquement les métadonnées.

thbar commented 8 months ago

Que penses-tu d'ajouter la contrainte de #36 ? Ou dans une version ultérieure si celle-ci gère uniquement les métadonnées.

C'est un bon rappel :-) Là je prévois plutôt de modifier uniquement la méta-donnée pour ne pas risquer d'impact non calculé (d'où le numéro de version 2.2.1 avec un changement uniquement du dernier numéro).

Il faut qu'on discute avec @cyrilmorin pour l'évolution du wattage, que ça soit concerté !

thbar commented 8 months ago

@etalab/transport-tech @etalab/transport-bizdev à votre bon coeur pour la review finale ; merci @AntoineAugusti pour les retours que j'ai pris en compte.

thbar commented 8 months ago

C'est traité, je ne l'avais pas fait car ça pose d'autres questions un peu plus profondes du type "que faire d'une adresse obsolète" etc. J'ai traité à minima il faudra y réfléchir plus avant (catch-all etc), car des personnes peuvent tenter de contacter à ces adresses et c'est dommage qu'elles "vieillissent mal".

Question pour plus tard, ça attendra bien.

  • Correctifs multiples #46 (comment) retirer les "mercis" pour garder un CHANGELOG simple et pertinent pour les producteurs de données IRVE (@johanricher et moi sommes ravis de tes mots 😅)

Boh je pense pas que c'était moins pertinent avec les mercis, mais pas de souci j'ai fait ça. Zou, on va pouvoir merger !

AntoineAugusti commented 8 months ago

Yep pour les contributeurs et les adresses ça peut attendre un peu et ça concerne tous les schémas de toute façon. Peut-être lister les noms des contributeurs et garder uniquement des adresses génériques d'équipes ?

schema@data.gouv.fr et contact@transport.data.gouv.fr par exemple.

AntoineAugusti commented 8 months ago

Merci et bien joué pour la mise à jour de nettoyage ! 🧹

thbar commented 8 months ago

Yep pour les contributeurs et les adresses ça peut attendre un peu et ça concerne tous les schémas de toute façon. Peut-être lister les noms des contributeurs et garder uniquement des adresses génériques d'équipes ?

Yes tout à fait j'en suis bien conscient ; ça peut rester une pensée dans un coin de notre tête pour le moment, pas de gros impact opérationnel, mais à réfléchir sur l'aspect pérennité longue durée.

Merci pour le merge, je vais pouvoir avancer et faire la release, puis faire les vérifications sur "schema.data.gouv.fr" derrière.

thbar commented 8 months ago

Merci et bien joué pour la mise à jour de nettoyage ! 🧹

Merki !