natemcmaster / LettuceEncrypt

Free, automatic HTTPS certificate generation for ASP.NET Core web apps
https://nuget.org/packages/LettuceEncrypt
Apache License 2.0
1.55k stars 153 forks source link

Update to C# 10 and adjust style to use file-scoped namespaces and implicit usings #231

Closed natemcmaster closed 2 years ago

natemcmaster commented 2 years ago

Now that VS 2022 and C# 10 are stable, update the code style to use two simplifying features -- file-scoped namespaces an implicit usings.

codecov[bot] commented 2 years ago

Codecov Report

Merging #231 (d52f4f4) into main (76a240d) will not change coverage. The diff coverage is 46.22%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #231   +/-   ##
=======================================
  Coverage   46.48%   46.48%           
=======================================
  Files          44       44           
  Lines        1108     1108           
=======================================
  Hits          515      515           
  Misses        593      593           
Impacted Files Coverage Δ
.../LettuceEncrypt/Internal/AcmeCertificateFactory.cs 0.00% <0.00%> (ø)
...c/LettuceEncrypt/Internal/AcmeCertificateLoader.cs 0.00% <0.00%> (ø)
src/LettuceEncrypt/Internal/AcmeClient.cs 0.00% <0.00%> (ø)
src/LettuceEncrypt/Internal/AcmeClientFactory.cs 0.00% <0.00%> (ø)
...ypt/Internal/AcmeStates/AcmeStateMachineContext.cs 0.00% <0.00%> (ø)
...ternal/AcmeStates/BeginCertificateCreationState.cs 0.00% <0.00%> (ø)
...ncrypt/Internal/AcmeStates/CheckForRenewalState.cs 0.00% <0.00%> (ø)
...eEncrypt/Internal/AcmeStates/ServerStartupState.cs 0.00% <0.00%> (ø)
...ettuceEncrypt/Internal/DomainOwnershipValidator.cs 0.00% <0.00%> (ø)
...c/LettuceEncrypt/Internal/Http01DomainValidator.cs 0.00% <0.00%> (ø)
... and 65 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 76a240d...d52f4f4. Read the comment docs.