initiative-sh / initiative.sh

A web-based command line for game masters
https://initiative.sh/
GNU General Public License v3.0
44 stars 4 forks source link

Fix rustfmt checking files twice #342

Closed MikkelPaulson closed 4 months ago

MikkelPaulson commented 4 months ago

Apparently rustfmt tests the entire code tree when pointed at lib.rs, so all files were being checked twice (except lib itself, I guess). The git ls-files song and dance was unnecessary, just point it at {lib,main}.rs and let 'er rip.

cloudflare-workers-and-pages[bot] commented 4 months ago

Deploying initiative-sh with  Cloudflare Pages  Cloudflare Pages

Latest commit: 86b3dbc
Status:⚡️  Build in progress...

View logs

MikkelPaulson commented 4 months ago
$ git ls-files '*.rs' | wc -l
139
$ rustfmt --check --edition 2021 --verbose */src/{lib,main}.rs */tests/*.rs | grep 'rs$' | wc -l
139