etalab / noms-de-domaine-organismes-secteur-public

Liste de noms de domaine d'organismes publics
Other
22 stars 15 forks source link

Merged data from adullact and ran consolidate.py #25

Closed JulienPalard closed 2 years ago

JulienPalard commented 2 years ago

Did the two in two different commits for readability:

I reviewed the 2nd commit to try to catch some issues, and found a few, see diff in sources/ in the 2nd commit, and additions to consolidate.py.

JulienPalard commented 2 years ago

(rebased on top of https://github.com/etalab/noms-de-domaine-organismes-publics/pull/27)

JulienPalard commented 2 years ago

ping @kuczynski ça te semble bon pour être mergé ? En regardant ce que je retire dans le 2ème commit tu peux peut être découvrir quelques domaines à retirer de ton côté aussi d'ailleurs.

bzg commented 2 years ago

Avec toi @JulienPalard on est trois à avoir validé, tu peux merger.

JulienPalard commented 2 years ago

Oui mais c'est le jeu de données de @kuczynski, ça me ferai plaisir d'avoir sa validation, je vais attendre un jour ou deux : je ne m'attendais pas forcément à une relecture en plein week-end.

Sinon je finirai par merger lundi ou mardi, oui, car cette branche n'est pas toujours triviale à rebaser.

bzg commented 2 years ago

N'hésite pas à mettre @mfaure en copie pour ce qui est des données d'ADULLACT.