search

jellyfin / jellyfin-plugin-kodisyncqueue

https://jellyfin.org
GNU General Public License v2.0
44 stars 13 forks source link

Add pre-commit config #41

Closed oddstr13 closed 3 years ago

oddstr13 commented 3 years ago

Run dotnet format

oddstr13 commented 3 years ago

Is this removing or adding the BOM? And I don't like the indentation change after the lock statement.

This is removing BOM, and I agree on the lock statement, if you know how to make dotnet format not do that that'd be great

oddstr13 commented 3 years ago

I couldn't figure out why it didn't add braces there, so I ended up adding them myself. The indentation it did is technically correct, as it did the equivalent of this;

if (true)
foo();

to

if (true)
    foo();

But it should also have wrapped it in {} there.