search

taxjar / taxjar.net

Sales Tax API Client for .NET / C#
https://developers.taxjar.com/api/reference/?csharp
MIT License
23 stars 23 forks source link

Keep whitespace consistent #33

Closed ScottRudiger closed 4 years ago

ScottRudiger commented 5 years ago

There was a mixture of tabs and (4) spaces for indentation in certain files. We're converting all tabs to (4) spaces in .cs files for consistency.

Going forward, an .editorconfig file will be in place to signal to IDEs our preferred settings for each file type.

File type Whitespace
*.cs and other file types 4 spaces
*.sln tabs (required)
.json, .csproj, .props, .targets 2 spaces
ScottRudiger commented 5 years ago

@fastdivision Just bumping this non-urgent PR in your notifications for when you have some extra time.

codymullins commented 4 years ago

@ScottRudiger is this still relevant?

ScottRudiger commented 4 years ago

@codymullins Yes, it just hasn't been urgent enough to push through. If you have some time, feel free to review and we can get it merged in! 🙏