FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc,crypto: mark experimental algorithms more visually by panva · Pull Request #44892 · nodejs/node · GitHub

/ node Public

doc,crypto: mark experimental algorithms more visually - #44892

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
panva:webcrypto-api-experimental
Oct 6, 2022
Merged

doc,crypto: mark experimental algorithms more visually#44892
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
panva:webcrypto-api-experimental

Conversation

panva commented Oct 4, 2022
edited
Loading

Copy link
Copy Markdown
Member

Similar to #44588 this adds a Stability index color-coded indicator to experimental WebCryptoAPI algorithms' documentation.

🖼️ Dark mode - table 🖼️ Light mode - table 🖼️ Dark mode - list 🖼️ Light mode - list

nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Oct 4, 2022
panva added crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations. experimental Issues and PRs related to experimental features. webcrypto and removed lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Oct 4, 2022
nodejs deleted a comment from nodejs-github-bot Oct 4, 2022

panva commented Oct 4, 2022

Copy link
Copy Markdown
Member Author

Review requested:

  • @nodejs/crypto
  • @nodejs/documentation

Trott commented Oct 5, 2022

Copy link
Copy Markdown
Member

I pulled these changes in and used a tool to check the resulting HTML for accessibility-related color contrast issues and there weren't any. 👍 Looks good to me.

panva marked this pull request as ready for review October 5, 2022 12:01
panva force-pushed the webcrypto-api-experimental branch from 6534dee to b9d74e0 Compare October 5, 2022 12:01
panva requested a review from Trott October 5, 2022 12:02
panva force-pushed the webcrypto-api-experimental branch from b9d74e0 to 555466e Compare October 5, 2022 12:11

aymen94 commented Oct 5, 2022

Copy link
Copy Markdown
Member

LGTM

panva added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Oct 5, 2022

Trott commented Oct 6, 2022

Copy link
Copy Markdown
Member

FYI for @nodejs/website @nodejs/nodejs-dev in case these kinds of things might affect the website in ways I'm not aware of.

ovflowd commented Oct 6, 2022

Copy link
Copy Markdown
Member

For nodejs.dev this change would not create any issues afaik.

Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 6, 2022
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 6, 2022
nodejs-github-bot merged commit aacd742 into nodejs:main Oct 6, 2022

Copy link
Copy Markdown
Collaborator

Landed in aacd742

danielleadams pushed a commit that referenced this pull request Oct 11, 2022
PR-URL: #44892
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
panva deleted the webcrypto-api-experimental branch October 13, 2022 09:12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations. experimental Issues and PRs related to experimental features. webcrypto

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL