Skip to content

Comments

lib: use validator#39536

Closed
VoltrexKeyva wants to merge 1 commit intonodejs:masterfrom
VoltrexKeyva:patch-3
Closed

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

Conversation

@VoltrexKeyva
Copy link
Member

@VoltrexKeyva VoltrexKeyva commented Jul 26, 2021

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

tniessen
tniessen reviewed Jul 27, 2021
View reviewed changes
@nodejs-github-bot 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
@VoltrexKeyva
lib: use validator
1096e16 
Used the `validateArray()` validator for validation of an array for consistency.
@VoltrexKeyva
Copy link
Member Author

VoltrexKeyva commented Jul 27, 2021

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

@VoltrexKeyva 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
@VoltrexKeyva
lib: use validator
3803072 
Used the `validateArray()` validator for array validation
for consistency. (Refs: nodejs#39536)
@VoltrexKeyva VoltrexKeyva mentioned this pull request Jul 27, 2021
Closed
jasnell pushed a commit that referenced this pull request Aug 12, 2021
@VoltrexKeyva @jasnell
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
@VoltrexKeyva @danielleadams
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
@VoltrexKeyva @targos
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tniessen 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

@VoltrexKeyva @tniessen @nodejs-github-bot