Skip to content

lib: cleanup validation#39652

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

lib: cleanup validation#39652
VoltrexKeyva wants to merge 1 commit intonodejs:masterfrom
VoltrexKeyva:patch-10

Conversation

@VoltrexKeyva
Copy link
Member

Used the validateInteger() validator to cleanup validation and keep
consistency.

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. tty Issues and PRs related to the tty subsystem. labels Aug 3, 2021
Used the `validateInteger()` validator to cleanup validation and keep
consistency.
@nodejs-github-bot
Copy link
Collaborator

@RaisinTen RaisinTen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Aug 8, 2021
@nodejs-github-bot
Copy link
Collaborator

@jasnell
Copy link
Member

jasnell commented Aug 12, 2021

Landed in ae0cfc9

@jasnell jasnell closed this Aug 12, 2021
@VoltrexKeyva VoltrexKeyva deleted the patch-10 branch August 12, 2021 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. needs-ci PRs that need a full CI run. tty Issues and PRs related to the tty subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants