Autre idée pour réparer le hook de pre-commit #626

Merged
mpepin merged 0 commits from Kerl/fix-pre-commit into master 2018-10-07 00:09:16 +02:00
mpepin commented 2018-10-06 23:51:58 +02:00 (Migrated from git.eleves.ens.fr)

On passe la liste des fichiers à checker à black, isort, etc avec xargs

Deux avantages :

  • On évite le globbing et word-splitting
  • Contrairement à mon patch précédent, ça marche 😇
On passe la liste des fichiers à checker à black, isort, etc avec `xargs` Deux avantages : - On évite le globbing et word-splitting - Contrairement à mon patch précédent, ça marche :innocent:
mpepin commented 2018-10-06 23:53:21 +02:00 (Migrated from git.eleves.ens.fr)

added ~63 ~64 ~59 ~16 ~28 labels

added ~63 ~64 ~59 ~16 ~28 labels
mpepin commented 2018-10-06 23:57:34 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • 6c815451 - remove useless parens

Compare with previous version

added 1 commit <ul><li>6c815451 - remove useless parens</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/322/diffs?diff_id=994&start_sha=fc4b852bde3cf30d96b4c428f02901fa3282460d)
mpepin commented 2018-10-07 00:07:28 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • 2c0ab1e5 - use xargs to prevent globbing in pre-commit.sh

Compare with previous version

added 1 commit <ul><li>2c0ab1e5 - use xargs to prevent globbing in pre-commit.sh</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/322/diffs?diff_id=995&start_sha=6c8154510e46ae2bafc1d1a35005c1afc584335f)
delobell commented 2018-10-07 00:09:08 +02:00 (Migrated from git.eleves.ens.fr)

Adjugé !

Adjugé !
delobell commented 2018-10-07 00:09:16 +02:00 (Migrated from git.eleves.ens.fr)

merged

merged
delobell commented 2018-10-07 00:09:16 +02:00 (Migrated from git.eleves.ens.fr)

mentioned in commit 26b19685b8

mentioned in commit 26b19685b8e0281375f81d88bbf2d5cf707bb837
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#626
No description provided.