Skip to content

Conversation

@binury
Copy link
Contributor

@binurybinury commented May 16, 2023

The forward slash of the comment on line 656 came loose.

ss 2023-05-16 at 14 53 19

This PR restores the structural integrity of the vertically-aligned column and improves resilience to undesired wavering.
ss 2023-05-16 at 14 53 35

Our OCD thanks you for your consideration of this tiny PR

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/url

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. url Issues and PRs related to the legacy built-in url module. labels May 16, 2023
@deokjinkim
Copy link
Contributor

@binury Thank you for your PR! Commit title should start with subsystem such as url. So please change your commit title to url: keep vertical alignment in noEscapeAuth.

https://github.com/nodejs/node/blob/main/doc/contributing/pull-requests.md#commit-message-guidelines

@panvapanva changed the title Clean: vertical alignment in url.jsurl: clean vertical alignment of docsMay 17, 2023
@deokjinkimdeokjinkim added request-ci Add this label to start a Jenkins CI on a PR. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels May 18, 2023
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 18, 2023
@nodejs-github-bot
Copy link
Collaborator

@deokjinkimdeokjinkim added the commit-queue Add this label to land a pull request using GitHub Actions. label May 18, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 18, 2023
@nodejs-github-botnodejs-github-bot merged commit 67b892e into nodejs:mainMay 18, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 67b892e

fasenderos pushed a commit to fasenderos/node that referenced this pull request May 22, 2023
PR-URL: nodejs#48037 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Deokjin Kim <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this pull request May 30, 2023
PR-URL: #48037 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Deokjin Kim <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@targostargos mentioned this pull request Jun 4, 2023
danielleadams pushed a commit that referenced this pull request Jul 6, 2023
PR-URL: #48037 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Deokjin Kim <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
MoLow pushed a commit to MoLow/node that referenced this pull request Jul 6, 2023
PR-URL: nodejs#48037 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Deokjin Kim <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.needs-ciPRs that need a full CI run.urlIssues and PRs related to the legacy built-in url module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@binury@nodejs-github-bot@deokjinkim@panva@lpinca@anonrig