Produits Entr'ouvert » Publik

Il y a 52 repos avec une config pre-commit. (Pourquoi pas 42 ?)

Il est possible que certains projets soit archivés. Je ne sais pas trop faire une liste exhaustive.

J'imagine qu'il faut surtout se concentrer sur celles utilisées dans public:

git.entrouvert.org/entrouvert/authentic.git
git.entrouvert.org/entrouvert/bijoe.git
git.entrouvert.org/entrouvert/chrono.git
git.entrouvert.org/entrouvert/combo.git
git.entrouvert.org/entrouvert/django-mellon.git
git.entrouvert.org/entrouvert/debian-django-tenant-schemas.git
git.entrouvert.org/entrouvert/eopayment.git
git.entrouvert.org/entrouvert/fargo.git
git.entrouvert.org/entrouvert/gadjo.git
git.entrouvert.org/entrouvert/hobo.git
git.entrouvert.org/entrouvert/lingo.git
git.entrouvert.org/entrouvert/passerelle.git
git.entrouvert.org/entrouvert/publik-base-theme.git
git.entrouvert.org/entrouvert/wcs.git
git.entrouvert.org/entrouvert/auquotidien.git
git.entrouvert.org/entrouvert/wcs-olap.git
git.entrouvert.org/entrouvert/welco.git

Je serais pour y aller partout (en réduisant à ceux qui ont un MANIFEST.in, ça devrait réduire d'une petite dizaine peut-être).

Avec le double test j'arrive à 38 repos. Ca devrait pouvoir servir de liste exaustive:

ants-hub
auquotidien
authentic
authentic2-auth-fedict
authentic2-cut
barbacompta
bijoe
chrono
combo
combo-plugin-gnm
combo-plugin-imio-townstreet
combo-plugin-nanterre
django-journal
django-mellon
docbow
eobuilder
eopayment
fargo
gadjo
godo.js
hobo
ldaptools
lingo
passerelle
passerelle-grandlyon-iodas
passerelle-imio-liege-lisrue
passerelle-minint
passerelle-montpellier-encombrants
passerelle-montpellier-sig
petale
pfwb-archives-proxy
publik-base-theme
publik-django-templatetags
scrutiny
wcs
wcs-olap
welco
xstatic-mapbox-gl-leaflet

Comme dit sur la PR passerelle, le fait d'avoir à ignorer des fichiers non versionnés est frustrant, mais ça fonctionne.

Par contre, c'est lent. 2.5s. Il y a un ticket qui explique pourquoi (tldr, ca lance deux sdist) https://github.com/mgedmin/check-manifest/issues/165

Je penses que ça va faire grogner tout le monde si c'est systématiquement lancé en pre-commit (moi le premier)

Je propose donc de faire lancer ça par tox/nox plutôt qu'en pre-commit. Ainsi ce sera lancé par jenkins et reproductible en locale. L'essentiel étant que le problème soit détecté avant le packaging.

Gael Pasgrimaud a écrit :

Je penses que ça va faire grogner tout le monde si c'est systématiquement lancé en pre-commit (moi le premier)

Je propose donc de faire lancer ça par tox/nox plutôt qu'en pre-commit. Ainsi ce sera lancé par jenkins et reproductible en locale. L'essentiel étant que le problème soit détecté avant le packaging.

Moi ça me va très bien comme ça mais au cas où: il y a d'autres "étapes" que pre-commit dans l'outil bizarrement nommé pre-commit, notamment pre-push que j'ai tenté d'utiliser pour faire passer la PR ruff sur eopayment (https://git.entrouvert.org/entrouvert/eopayment/pulls/17/files) qui achoppe malheureusement sur d'autres objections que le fait que soit pre-push ou pre-commit.

J'ai fait deux PR de principe.
Une sur eobuilder avec tox: https://git.entrouvert.org/entrouvert/eobuilder/pulls/6
Une sur passerelle avec nox: https://git.entrouvert.org/entrouvert/passerelle/pulls/564

Etant donné que des fichiers sont ajoutés au build, j'ai rajouté une étape "python setup.py build sdist" qui garantit qu'on génère bien tout avant l'analyse.

Ca semble ok

Je note lors de la relecture du patch pour w.c.s. le graft debian qui va intégrer les répertoires debian/ dans les .tar.gz; pour moi il ne faut pas faire ça.

Some projects include a rough /debian directory among source files to ease bleeding-edge package compilation and installation on Debian (and derived) systems. While this is a good effort, it is better to leave it out of the final tarball as it can interfere with debian's own packaging effort.
— https://wiki.debian.org/UpstreamGuide

Ca se défends. Je vais trouver le sed qui va bien pour changer ça partout (et je note que c'était quand même un bon effort :D)

Fait. A part dans passerelle, qui avait déjà été validé / mergé. Je passerai un commit sur main pour ce détail plus tard si c'est ok

Note que tu peux lier les PR directement à ce ticket plutôt que de créer un ticket dans chaque brique, voire même les merger sans relecture (en annonçant à l'avance qu'une validation sur une PR vaut validation sur toutes)

Je notes mais j'aime autant faire un ticket par projet. Je ne serai pas plus efficace en sortant du workflow habituel (désolé pour le trafic "inutile" que ça engendre)

Et oui, je pensais bien demander lundi si ok pour merger sans relecture. C'est toujours pareil, un peu inutile à relire maintenant que quelques une sont validées