8000 Use file scoped namespaces by iamcarbon · Pull Request #343 · passwordless-lib/fido2-net-lib · GitHub
[go: up one dir, main page]

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
0