You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Biome step empties files when the file is ignored since biome 1.5.0 #1987
The biome step uses the following command to format files via biome:
biome format --stdin-file-path [fileName] --config-path [PathToBiomeConfig] < [InputFile]
The biome.json config file can contain an ignore field with files that should not be processed. When the given file name matches that ignore list, biome won't format the file. In biome versions <1.5.0, it would still output the input file unchanged to stdout, and log a warning to stderr.
Since version 1.5.0, biome does not output anything to stdout, and only print a warning to stderr
⚠ The file localization.ts was ignored.
As a result, when trying to format a file via the biome step that is ignored, the file will be emptied, without the user noticing.
I would consider this a misconfiguration, as the includes/excludes defined for the spotless plugin conflict with the includes from the biome configuration. Still, it's not good to destroy data without the user being aware that something went wrong (I accidentally committed something, only to realize this later).
Ideally, we would just ignore the include/exclude list from the biome configuration. Since the biome CLI doesn't have such a flag, my suggestion is to check whether the biome output is empty; and if it is, use the original file content instead. In addition, we could either output a warning on our own, or forward the warning from the stderr stream from biome (if that's not already being done).
There's also a different scenario where biome refuses to format certain files, such as tsconfig.json. Since version 1.5.0, this results in the same issue that stdout is empty, resulting in the formatter emptying the file
ℹ The file tsconfig.json is protected because is handled by another tool. Biome won't process it.
Activity
blutorange commentedon Jan 9, 2024
There's also a different scenario where biome refuses to format certain files, such as
tsconfig.json. Since version 1.5.0, this results in the same issue that stdout is empty, resulting in the formatter emptying the fileFixes diffplug#1987, no empty files with biome when formatting ignore…
no empty files with biome when formatting ignored files (#1989 fixes #…
nedtwigg commentedon Jan 15, 2024
Published in
plugin-maven 2.42.0andplugin-gradle 6.24.0.chore(deps): update all non-major dependencies (mulk/mulkcms2!58)