refactor: --check flag and manifest in any order #17

Merged
squel merged 2 commits from squel/cli-2 into main 2025-03-13 21:35:12 +00:00 AGit
Owner

drop urfave/cli seems limited by Go's upstream
flag package:

Flag parsing stops just before the first
non-flag argument ("-" is a non-flag argument) or
after the terminator "--".
https://pkg.go.dev/flag#hdr-Command_line_flag_syntax

drop urfave/cli seems limited by Go's upstream flag package: > Flag parsing stops just before the first non-flag argument ("-" is a non-flag argument) or after the terminator "--". https://pkg.go.dev/flag#hdr-Command_line_flag_syntax
squel changed title from refactor: drop --check flag and manifest in any order to refactor: --check flag and manifest in any order 2025-02-13 11:29:37 +00:00
squel force-pushed squel/cli-2 from 181785b2a0 to 4dc670aeba 2025-02-13 11:30:35 +00:00 Compare
squel force-pushed squel/cli-2 from 4dc670aeba to cc448c276d 2025-02-13 11:56:11 +00:00 Compare
Author
Owner

note on windows users:

  • cmd.exe doesnt parse color escape codes, so they should run set NO_COLOR=1 before executing
  • cmd.exe doesnt glob files, so incase of a multi-part .manifest they need to be run one-by-one
note on windows users: - `cmd.exe` doesnt parse color escape codes, so they should run `set NO_COLOR=1` before executing - `cmd.exe` doesnt glob files, so incase of a multi-part `.manifest` they need to be run one-by-one
summary messages are printed with a green "OK"
prefix, like:
- OK all (n) checked files OK! (all hashes match)
- OK all (n) on disk have been checked!
- OK all (n) on defined in the minput manifest
  have been checked!

also refactor ansi to use an internal helper
function lol
squel merged commit 795216f7fb into main 2025-03-13 21:35:12 +00:00
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
squel/steam-manifest!17
No description provided.