Liblor / applied_sec_lab

Applied Security Laboratory - AS19
6 stars 1 forks source link

Scaffold user-facing web server ASP.NET Core project #1

Closed RequestForCoffee closed 5 years ago

keyctl commented 5 years ago

This works on Fedora 30 with Microsofts proprietary dotnet-3.0 package.

keyctl commented 5 years ago

@RequestForCoffee Do you think we could switch to using tabs?

RequestForCoffee commented 5 years ago

@RequestForCoffee Do you think we could switch to using tabs?

I think the Visual Studio 'default' for C# is 4-space indentation; what would be the benefit of tabs here?

keyctl commented 5 years ago

@RequestForCoffee Do you think we could switch to using tabs?

I think the Visual Studio 'default' for C# is 4-space indentation; what would be the benefit of tabs here?

Benefit being that with tabs everyone can chose the indentation width they like. In contrast, with space one is always bound to the width the author provides.

RequestForCoffee commented 5 years ago

@RequestForCoffee Do you think we could switch to using tabs?

I think the Visual Studio 'default' for C# is 4-space indentation; what would be the benefit of tabs here?

Benefit being that with tabs everyone can chose the indentation width they like. In contrast, with space one is always bound to the width the author provides.

I guess we could just agree on a width. Then again, I wouldn't really mind you reformatting it to tabs.

keyctl commented 5 years ago

Other than that, let's merge it!