fix(nix-reuse): Run precommit hook only on changed files #1

Closed

mdebray wants to merge 1 commit from fix_greedyness into main

mdebray commented

2025-01-23 09:41:05 +01:00

Owner

No description provided.

mdebray added 1 commit 2025-01-23 09:41:05 +01:00

fix(nix-reuse): Run precommit hook only on changed files fcdbf37b7d

mdebray referenced this pull request from DGNum/infrastructure

2025-01-23 09:44:22 +01:00

REUSE hook is too greedy and should ignore untracked files #232

mdebray referenced this pull request from DGNum/infrastructure

2025-01-23 09:45:49 +01:00

feat(reuse): Switch to an action rather than a git hook, to be less cumbersome #262

thubrecht commented

2025-01-23 12:58:52 +01:00

Owner

Y'a plusieurs choses qui vont pas:

Tu modifies juste le nom du hook ? Si tu changes pas l'entry, je vois pas pourquoi ça donnerait un autre résultat
Imaginons que tu changes aussi l'entry, le hook ne passe pas en argument les noms des fichiers modifiés
Imaginons que les fichiers modifiés soient correctement donnés, ça ne fonctionne pas si la modification porte sur le fichier REUSE.toml, qui ferait que le dépôt ne soit plus compliant

Y'a plusieurs choses qui vont pas: - Tu modifies juste le nom du hook ? Si tu changes pas l'entry, je vois pas pourquoi ça donnerait un autre résultat - Imaginons que tu changes aussi l'entry, le hook ne passe pas en argument les noms des fichiers modifiés - Imaginons que les fichiers modifiés soient correctement donnés, ça ne fonctionne pas si la modification porte sur le fichier REUSE.toml, qui ferait que le dépôt ne soit plus compliant

mdebray commented

2025-01-23 23:38:48 +01:00

Author

Owner

Point 1 oui my bad c'est le deuxième
Point 2 non les fichiers sont passé par défaut
Point 3 effectivement il faut être plus subtil. Je regarde

Point 1 oui my bad c'est le deuxième Point 2 non les fichiers sont passé par défaut Point 3 effectivement il faut être plus subtil. Je regarde

mdebray commented

2025-01-24 00:05:37 +01:00

Author

Owner

Pour le point 3, on va dire que on réduit effectivement les trucs testés mais on passe pas à 0 comme proposé dans la PR de l'infra. Je modifie pas les actions avec mon commit il me semble.

Pour le point 2 je pousse une branche "mdebray/reuse_fix" sur l'infra pour montrer

Pour le point 3, on va dire que on réduit effectivement les trucs testés mais on passe pas à 0 comme proposé dans la PR de l'infra. Je modifie pas les actions avec mon commit il me semble. Pour le point 2 je pousse une branche "mdebray/reuse_fix" sur l'infra pour montrer

mdebray force-pushed fix_greedyness from fcdbf37b7d to aa1bb4685e

2025-01-24 00:05:45 +01:00

Compare

mdebray closed this pull request

2025-01-24 00:08:31 +01:00