Repository navigation
Conversation
mathiasrw
added this pull request to stack #787
October 23, 2026 04:29
# Conflicts: # README.md
…with names that differ only in case
# Conflicts: # test/cli/fix/742.test.js # test/fix/783.test.js
… record only the renames that were done
…ith small examples in the README
# Conflicts: # README.md
# Conflicts: # README.md # devguide.md # test/cli/debug.test.js # test/cli/delimiter.test.js # test/cli/glob-pipe.test.js # test/cli/run.sh # test/engine/encoding.test.js # test/engine/halt.test.js # test/fix/681.test.js # test/fix/683.test.js # test/fix/691.test.js # test/fix/700.test.js # test/fix/703.test.js # test/fix/708.test.js # test/fix/720.test.js # test/fix/728.test.js # test/fix/742.test.js
mathiasrw
marked this pull request as draft
October 23, 2026 09:45
# Conflicts: # README.md # src/cli.ts # src/engine.ts # test/cli/run.sh # test/engine/rename.test.js # test/helper.js
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #782
Stacked on #785.
Change
--expect-files-F--expect-diff-z--expect-zero-diff-Z--expect-diff-allWhen a check is not met, the exit code is 1. The run is as without the flags, so add
--simulateto only check.wouldwith--simulate,--outputand--output-match, andwasorwerewithout.--expect-diffand--expect-diff-allare not met, and--expect-zero-diffis.--expect-filesmakes no files fail too.--quiet, and it is inerrorsin the status report, soerrors.lengthtells if anything was wrong. It is also inchecks, so the CLI gives 1 when the failed checks are the only errors.--expect-zero-diffwith--expect-diffor--expect-diff-allis an error in the CLI (exit code 3), and a panic in the engine.Also in this PR
--replacement-jsuse its shorter name--js.r()(Add --js-file to read the code from a file #793).Tests
yarn test-cipasses: the demo on Node and Bun (now with--expect-*), and 969 Bun tests. There is one engine test file for each flag, andtest/cli/expect.test.js.