Skip to content

Simplify enum mapping & update specification links - #368

Merged
abergs merged 3 commits into
passwordless-lib:masterfrom
carbon:enums
Jun 22, 2023
Merged

abergs merged 3 commits into
passwordless-lib:masterfrom
carbon:enums

Conversation

@iamcarbon

Copy link
Copy Markdown
Contributor

This PR:

  • Removes support for enforcing the case sensitivity of the Enum parser. Enum parsing has always been case insensitive in the library.

  • Update various references to a draft webauthn specification to the final published specification.

@iamcarbon

Copy link
Copy Markdown
Contributor Author

@aseigler @abergs Ready for review.

@abergs abergs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Jason, this looks good!

@iamcarbon

Copy link
Copy Markdown
Contributor Author

@abergs Let me know if there's anything else here before we merge.

@iamcarbon

Copy link
Copy Markdown
Contributor Author

@abergs Let me know if this is good to merge.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #368 (08a1a65) into master (7b92f4a) will decrease coverage by 0.14%.
The diff coverage is 70.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #368      +/-   ##
==========================================
- Coverage   74.70%   74.56%   -0.14%     
==========================================
  Files          92       92              
  Lines        2728     2721       -7     
  Branches      456      454       -2     
==========================================
- Hits         2038     2029       -9     
- Misses        577      578       +1     
- Partials      113      114       +1     
Impacted Files Coverage Δ
Src/Fido2.Models/CredentialCreateOptions.cs 41.86% <0.00%> (ø)
Src/Fido2/Extensions/EnumExtensions.cs 66.66% <50.00%> (-33.34%) ⬇️
Src/Fido2.Models/Converters/EnumNameMapper.cs 100.00% <100.00%> (ø)

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