diff options
| author | ojacobson <ojacobson@noreply.codeberg.org> | 2025-06-10 00:43:57 +0200 |
|---|---|---|
| committer | ojacobson <ojacobson@noreply.codeberg.org> | 2025-06-10 00:43:57 +0200 |
| commit | 71919b461f25bed1bb4708b0494d603de373ae11 (patch) | |
| tree | c6fa8e3f4dc6200dbda83fe5257b5631f59bed22 /git-hooks/pre-commit | |
| parent | 2273b6dce6cb650c0a72b71491421f198518bfb7 (diff) | |
| parent | bf1ef354817dba47cc955e9654dc204f0dc07d27 (diff) | |
Rework project formatting tools.
* Changes JS formatting policy to expect trailing commas in most of the places they're allowed.
* Moves format checking out of `npm` and into tool scripts.
* _Documents_ formatting tools, as well as a shot at a policy for format-only changes designed to alleviate low-value review.
This was originally part of [another proposal][pr-6]. I've broken it out to make the intent clearer, and to make the proposal easier to get a handle on in isolation from other, related changes. Thanks to @wlonk for their input on this!
[pr-6]: https://codeberg.org/ojacobson/pilcrow/pulls/6
Merges prop/project-formatting into main.
Diffstat (limited to 'git-hooks/pre-commit')
| -rwxr-xr-x | git-hooks/pre-commit | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/git-hooks/pre-commit b/git-hooks/pre-commit index 587e349..3d0ae17 100755 --- a/git-hooks/pre-commit +++ b/git-hooks/pre-commit @@ -1,8 +1,9 @@ #!/bin/bash -e # Don't put anything here that routinely takes longer than a second or so to -# run. It gets old fast. That's why this uses `cargo check` and not `cargo -# test`, for example. +# run. It gets old fast. That's why this doesn't run tests, for example. + +tools/check-format # Make sure package-lock.json is up to date with package.json npm ci --dry-run |