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

lib: use validator by VoltrexKeyva · Pull Request #39536 · nodejs/node · GitHub

/ node Public

lib: use validator - #39536

Closed
VoltrexKeyva wants to merge 1 commit into
nodejs:masterfrom
VoltrexKeyva:patch-3
ConversationCommits (1)ChecksFiles changed
Closed

lib: use validator#39536
VoltrexKeyva wants to merge 1 commit into
nodejs:masterfrom
VoltrexKeyva:patch-3

Conversation

VoltrexKeyva commented Jul 26, 2021

Copy link
Copy Markdown
Contributor

Used the validateArray() validator for validation of an array for
consistency.

tniessen reviewed Jul 27, 2021
View reviewed changes
Comment thread lib/internal/child_process.js Outdated
nodejs-github-bot added child_process Issues and PRs related to the child_process subsystem. needs-ci PRs that need a full CI run. labels Jul 27, 2021
lib: use validator
1096e16 
Used the `validateArray()` validator for validation of an array for consistency.

VoltrexKeyva commented Jul 27, 2021

Copy link
Copy Markdown
Contributor Author

Seems like something went wrong here, re-opening with a new PR.

VoltrexKeyva deleted the patch-3 branch July 27, 2021 15:59
VoltrexKeyva added a commit to VoltrexKeyva/node that referenced this pull request Jul 27, 2021
lib: use validator
3803072 
Used the `validateArray()` validator for array validation
for consistency. (Refs: nodejs#39536)
VoltrexKeyva mentioned this pull request Jul 27, 2021
Closed
jasnell pushed a commit that referenced this pull request Aug 12, 2021
lib: use validator
c3b4d3f 
Used the `validateArray()` validator for array validation
for consistency. (Refs: #39536)

PR-URL: #39547
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
danielleadams pushed a commit that referenced this pull request Aug 16, 2021
lib: use validator
be2d60d 
Used the `validateArray()` validator for array validation
for consistency. (Refs: #39536)

PR-URL: #39547
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
targos pushed a commit that referenced this pull request Sep 4, 2021
lib: use validator
9c33e4b 
Used the `validateArray()` validator for array validation
for consistency. (Refs: #39536)

PR-URL: #39547
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

tniessen tniessen left review comments

Assignees

No one assigned

Labels

child_process Issues and PRs related to the child_process subsystem. needs-ci PRs that need a full CI run.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL