Skip to content

lib: refactor to use validateFunction #37045

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
Lxxyx wants to merge 2 commits into from
Closed

lib: refactor to use validateFunction #37045

Lxxyx wants to merge 2 commits into from

Conversation

@Lxxyx
Copy link
Member

@Lxxyx Lxxyx commented Jan 24, 2021

  1. add a new validator: validateFunction
  2. refactor to use validateFunction

@Lxxyx
lib: refactor to use validateFunction
f3842ee 
add validateFunction and refactor to use validateFunction
@nodejs-github-bot nodejs-github-bot added the lib / src Issues and PRs related to general changes in the lib or src directory. label Jan 24, 2021
@Lxxyx
Copy link
Member Author

Lxxyx commented Jan 24, 2021 •
edited
Loading

validateFunction and validateCallback are similar, but ERR_INVALID_ARG_TYPE is more widely used and more accurate, maybe we should refactor validateCallback to validateFunction in the next major release? @Trott

Refs: #36609 (review)

aduh95
aduh95 approved these changes Jan 24, 2021
View reviewed changes
@Lxxyx @aduh95
Update lib/async_hooks.js
64ba104 
Co-authored-by: Antoine du Hamel <[email protected]>
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jan 24, 2021

CI: https://ci.nodejs.org/job/node-test-pull-request/35702/

@RaisinTen RaisinTen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 24, 2021
@benjamingr
Copy link
Member

benjamingr commented Jan 27, 2021

Landed in 064a707 🎉

@benjamingr benjamingr closed this Jan 27, 2021
benjamingr pushed a commit that referenced this pull request Jan 27, 2021
@Lxxyx @benjamingr
lib: refactor to use validateFunction
064a707 
add validateFunction and refactor to use validateFunction

PR-URL: #37045
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
targos pushed a commit that referenced this pull request Feb 2, 2021
@Lxxyx @targos
lib: refactor to use validateFunction
5227c5e 
add validateFunction and refactor to use validateFunction

PR-URL: #37045
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@targos targos mentioned this pull request Feb 2, 2021
Merged
This was referenced Jul 16, 2021
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@Trott Trott Trott approved these changes

@lpinca lpinca lpinca approved these changes

@aduh95 aduh95 aduh95 approved these changes

@RaisinTen RaisinTen RaisinTen approved these changes

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. lib / src Issues and PRs related to general changes in the lib or src directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@Lxxyx @nodejs-github-bot @benjamingr @jasnell @Trott @lpinca @aduh95 @RaisinTen @targos