search

davidfowl / TodoApi

Todo application with ASP.NET Core Blazor WASM, Minimal APIs and Authentication
MIT License
2.79k stars 419 forks source link

use implicit types where obvious #34

Closed SimonCropp closed 1 year ago

cwe1ss commented 1 year ago

To me, only the "new Claim() -> new()" is (kind of) obvious - the others aren't. E.g., the implicit new Uri() for the BaseAddress is only obvious if you have good knowledge of that API.

I don't think this would be a good change for this showcase project.

davidfowl commented 1 year ago

Agree, I'm not loving this change.

SimonCropp commented 1 year ago

@davidfowl yeah i thought this one might be controversial. it is more on the personal preference side of things. will close this one