From 0cd64c87bf72f9237eedb9bdc9739411be9120cb Mon Sep 17 00:00:00 2001 From: Braden Kelley Date: Thu, 20 Jun 2024 16:19:55 -0700 Subject: [PATCH] fix(biome): use temp file instead of stdin (#4775) biome handles utf8 characters differently between files and stdin, and in some cases can replace emojis with ascii characters when using stdin refs: biomejs/biome#2604 --- autoload/ale/fixers/biome.vim | 3 ++- test/fixers/test_biome_fixer_callback.vader | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/autoload/ale/fixers/biome.vim b/autoload/ale/fixers/biome.vim index 0f210657..c16cb3b8 100644 --- a/autoload/ale/fixers/biome.vim +++ b/autoload/ale/fixers/biome.vim @@ -3,8 +3,9 @@ function! ale#fixers#biome#Fix(buffer) abort let l:options = ale#Var(a:buffer, 'biome_options') return { + \ 'read_temporary_file': 1, \ 'command': ale#Escape(l:executable) . ' format' \ . (!empty(l:options) ? ' ' . l:options : '') - \ . ' --stdin-file-path=%s', + \ . ' %t' \} endfunction diff --git a/test/fixers/test_biome_fixer_callback.vader b/test/fixers/test_biome_fixer_callback.vader index 632a7ec4..4c417394 100644 --- a/test/fixers/test_biome_fixer_callback.vader +++ b/test/fixers/test_biome_fixer_callback.vader @@ -9,7 +9,8 @@ Execute(The default biome command should be correct): AssertFixer \ { + \ 'read_temporary_file': 1, \ 'command': ale#Escape('biome') - \ . ' format --stdin-file-path=%s' + \ . ' format %t' \ }