craftsman icon indicating copy to clipboard operation
craftsman copied to clipboard

Generated code contains both tabs and spaces

Open vde-developer opened this issue 3 years ago • 2 comments

When using craftsman, generated classes contain a mix of tabs and spaces. Yes i know it doesn't matter functionally, and i don't have an opinion on tabs or spaces, just the mix of both can cause empty diffs in some diff software.

image

Craftsman version: 0.15.0

vde-developer avatar Jul 18 '22 15:07 vde-developer

Yeah there should be consistency here. Not sure when I'll get to this, but noted and I'll try to get to this.

If you want to submit a PR to fix them or at least some of the higher traffic classes, I'll accept it.

In the meantime, I'd recommend a dotnet format to fix them after a scaffold and before a commit.

pdevito3 avatar Jul 18 '22 15:07 pdevito3

Thanks about the dotnet format tip!

I've checked the behavior of generating classes with Craftsman after replacing all tabs with 4 spaces, and it seems to work as for me... But i honestly haven't had enough experience using it to know if I'm missing anything.

If anyone with more experience with Craftsman want to look I've created a pull request. #88

vde-developer avatar Jul 18 '22 16:07 vde-developer