Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File report on Enum values that ignore naming conventions in Web Cryptography API #712

Merged
merged 3 commits into from
Sep 9, 2024

Conversation

dontcallmedom-bot
Copy link
Contributor

This pull request was automatically created by Strudy upon detecting errors in Enum values that ignore naming conventions in Web Cryptography API.

Please check that these errors were correctly detected, and that they have not already been reported in https://github.com/w3c/webcrypto.

If everything is OK, you can merge this pull request which will report the issue below to the repo, and update the underlying report file with a link to the said issue.


Title: Enum values that ignore naming conventions in Web Cryptography API
Tracked: N/A
Repo: 'https://github.com/w3c/webcrypto'

While crawling Web Cryptography API, the following enum values were found to ignore naming conventions (lower case, hyphen separated words):

  • The value "deriveKey" of the enum KeyUsage does not match the expected conventions (lower case, hyphen separated words)
  • The value "deriveBits" of the enum KeyUsage does not match the expected conventions (lower case, hyphen separated words)
  • The value "wrapKey" of the enum KeyUsage does not match the expected conventions (lower case, hyphen separated words)
  • The value "unwrapKey" of the enum KeyUsage does not match the expected conventions (lower case, hyphen separated words)

See Use casing rules consistent with existing APIs in Web Platform Design Principles document for guidance.

This issue was detected and reported semi-automatically by Strudy based on data collected in webref.

@tidoust
Copy link
Member

tidoust commented Sep 9, 2024

I added the "Cc" paragraph to hopefully avoid it being raised another time!

@tidoust tidoust merged commit 7b83697 into main Sep 9, 2024
1 check passed
@tidoust tidoust deleted the WebCryptoAPI-wrongcaseenumvalue branch September 9, 2024 10:29
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