Skip to content

doc: use kbd element in readline doc prose - #35737

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:one-moar-kbd
Oct 26, 2020
Merged

doc: use kbd element in readline doc prose#35737
Trott merged 1 commit into
nodejs:masterfrom
Trott:one-moar-kbd

Conversation

@Trott

Copy link
Copy Markdown
Member

The kbd element has already been added in the keybindings table, but we
missed it in one line in the prose.

Checklist

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. readline Issues and PRs related to the built-in readline module. labels Oct 21, 2020
@Trott

Copy link
Copy Markdown
MemberAuthor

Before:

image

After:

image

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #35737 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #35737 +/- ##
==========================================
- Coverage 96.40% 96.40% -0.01% 
==========================================
Files 223 223 Lines 73685 73685 ==========================================
- Hits 71038 71035 -3 - Misses 2647 2650 +3 
Impacted FilesCoverage Δ
lib/_http_server.js98.34% <0.00%> (-0.31%)⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c4239b...8234f2d. Read the comment docs.

The kbd element has already been added in the keybindings table, but we
missed it in one line in the prose.
PR-URL: nodejs#35737
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@Trott
Trott merged commit dd43715 into nodejs:masterOct 26, 2020
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in dd43715

@Trott
Trott deleted the one-moar-kbd branch October 26, 2020 10:34
targos pushed a commit that referenced this pull request Nov 3, 2020
The kbd element has already been added in the keybindings table, but we
missed it in one line in the prose.
PR-URL: #35737
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@targostargos mentioned this pull request Nov 3, 2020
BethGriggs pushed a commit that referenced this pull request Dec 8, 2020
The kbd element has already been added in the keybindings table, but we
missed it in one line in the prose.
PR-URL: #35737
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
BethGriggs pushed a commit that referenced this pull request Dec 10, 2020
The kbd element has already been added in the keybindings table, but we
missed it in one line in the prose.
PR-URL: #35737
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Dec 10, 2020
BethGriggs pushed a commit that referenced this pull request Dec 15, 2020
The kbd element has already been added in the keybindings table, but we
missed it in one line in the prose.
PR-URL: #35737
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.readlineIssues and PRs related to the built-in readline module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Trott@codecov-io@fhinkel@silverwind@nodejs-github-bot