search

Code2Gether-Discord / Congo

Generic web store learning project that consists of three ASP.NET Core projects
https://code2gether-congo.azurewebsites.net/
11 stars 2 forks source link

.NET 6 Migration #97

Open U-C-S opened 2 years ago

U-C-S commented 2 years ago

Discussion and Tracking Issue to migrate to .NET 6 (from .NET 5)

Important Links

pobiega commented 2 years ago

Great initiative.

I suggest that as part of this story, we also...

  1. Switch to file-scoped namespaces, to reduce indentation.
  2. Activate implicit usings, to reduce repeat using statements.
  3. Do not swap to top level statements in Program.cs.
  4. Do not swap to minimal APIs.

If anyone disagrees with any of these points, please comment below.

valincius commented 2 years ago

Do we want to enable nullable reference types? Will make a bit more work but net6 projects have this enabled by default, so I think we should do it.

pobiega commented 2 years ago

enable nullable reference types?

I'm fine with it, but I suggest we do it as a separate PR, as the file scope namespaces will make the diff near unreadable :)

Palisar commented 2 years ago

How do we feel about having each project have a Usings.cs that contains all of our global usings ?