flake8-builtins icon indicating copy to clipboard operation
flake8-builtins copied to clipboard

make black and isort mandatory tests

Open cclauss opened this issue 2 years ago • 2 comments

black --skip-string-normalization .
isort --profile black .

cclauss avatar Sep 20 '21 11:09 cclauss

Thanks! but as you see I very much follow black completely but I have a strong preference for single quotes. Fortunately black has an option to ignore those skip-string-normalization would you mind adding that so that the diff is veeeeeery much reduced? TIA!! 🙇🏾

gforcada avatar Sep 20 '21 16:09 gforcada

thanks! would you mind squashing both commits together? so that history/blame later on is not confused by them? 👍🏾

gforcada avatar Sep 21 '21 07:09 gforcada

@gforcada any chance you could merge this? I have to disable auto-formatting in my editor while making code changes, as it's adding in a bunch of unrelated black/isort changes as well.

gsingh93 avatar Oct 06 '22 00:10 gsingh93

@cclauss thanks for the changes again, and @gsingh93 for the ping, I'm rather busy these days 😅 merging! 😄

gforcada avatar Oct 07 '22 11:10 gforcada