fix(core): list only changed files on format:write log #10155
Merged
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.
Current Behavior
The
format:write
currently includes explicit checks onnx.json
,workspace.json
andtsconfig.base.json
and performs write on all of them including all the files matching the provided pattern.This results in a log containing all of the files touched, although in reality only a few of them are actually changed.
The
format:check
lists only files that require fixing.Expected Behavior
The
format:write
should list only files that were changed during the formatting, not all of the files being checked.This is useful for using
format:write
as part of the pre-commit hook and automatically add changed files to commit.Related Issue(s)
Fixes #