Skip to content

doc: fix and modernize code examples in crypto.md#10909

Closed
vsemozhetbyt wants to merge 8 commits intonodejs:masterfrom
vsemozhetbyt:crypto.md
Closed

doc: fix and modernize code examples in crypto.md#10909
vsemozhetbyt wants to merge 8 commits intonodejs:masterfrom
vsemozhetbyt:crypto.md

Conversation

@vsemozhetbyt
Copy link
Contributor

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc, crypto

  • var -> const / let.
  • Fix error in code example (basing on the previous example).
  • equal -> strictEqual, == -> ===.
  • Update estimated outputs in code example comments.
  • snake_case -> camelCase (unlike method signatures, examples mostly use camelCase).
  • Concatenation -> multiline template.
  • Add missing line break in code example (the code throws currently; fixed basing on the history).
  • Add missing link reference.

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

Labels

crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants