KEMBAR78
Use file scoped namespaces by iamcarbon · Pull Request #343 · passwordless-lib/fido2-net-lib · GitHub
Skip to content

Conversation

@iamcarbon
Copy link
Contributor

This PR converts the remaining files to use file-scoped namespaces. There are no functional changes in this PR.

@iamcarbon
Copy link
Contributor Author

@abergs @aseigler Ready for review / feedback.

@iamcarbon
Copy link
Contributor Author

Consider reviewing while ignoring whitespace.

https://github.com/passwordless-lib/fido2-net-lib/pull/343/files?diff=split&w=1

@codecov-commenter
Copy link

Codecov Report

Merging #343 (61a3590) into master (dd727b1) will not change coverage.
The diff coverage is 77.69%.

@@           Coverage Diff           @@
##           master     #343   +/-   ##
=======================================
  Coverage   77.87%   77.87%           
=======================================
  Files          88       88           
  Lines        2495     2495           
  Branches      415      415           
=======================================
  Hits         1943     1943           
  Misses        439      439           
  Partials      113      113           
Impacted Files Coverage Δ
Src/Fido2.AspNet/Fido2NetLibBuilderExtensions.cs 0.00% <0.00%> (ø)
Src/Fido2.AspNet/NullMetadataService.cs 0.00% <0.00%> (ø)
Src/Fido2.Models/Metadata/BiometricStatusReport.cs 0.00% <0.00%> (ø)
Src/Fido2.Models/Metadata/EcdaaTrustAnchor.cs 0.00% <0.00%> (ø)
...o2.Models/Objects/PublicKeyCredentialUserEntity.cs 0.00% <0.00%> (ø)
Src/Fido2.Models/Objects/Version.cs 0.00% <0.00%> (ø)
Src/Fido2/Attributes/Fido2StandardAttribute.cs 0.00% <0.00%> (ø)
Src/Fido2/ConformanceMetadataService.cs 0.00% <0.00%> (ø)
Src/Fido2/DevelopmentInMemoryStore.cs 0.00% <0.00%> (ø)
Src/Fido2/Metadata/FileSystemMetadataRepository.cs 0.00% <0.00%> (ø)
... and 79 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@abergs abergs merged commit 9823ec0 into passwordless-lib:master Nov 24, 2022
@abergs
Copy link
Collaborator

abergs commented Nov 24, 2022

LGTM 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants