Skip to content

lib: refactor to use validators in http2#46174

Merged
nodejs-github-bot merged 4 commits intonodejs:mainfrom
debadree25:ft/refactor-to-validate-boolean
Jan 24, 2023
Merged

lib: refactor to use validators in http2#46174
nodejs-github-bot merged 4 commits intonodejs:mainfrom
debadree25:ft/refactor-to-validate-boolean

Conversation

@debadree25
Copy link
Copy Markdown
Member

@debadree25 debadree25 commented Jan 11, 2023

Attempted to refactor usage of validators similar to the referenced PR

Refs: #46101

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@VoltrexKeyva VoltrexKeyva VoltrexKeyva left review comments

@mcollina mcollina mcollina approved these changes

@ShogunPanda ShogunPanda ShogunPanda approved these changes

@lpinca lpinca lpinca approved these changes

@JungMinu JungMinu JungMinu approved these changes

@trivikr trivikr trivikr approved these changes

@RafaelGSS RafaelGSS RafaelGSS approved these changes

Assignees

No one assigned

Labels

commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. http2 Issues or PRs related to the http2 subsystem. semver-major PRs that contain breaking changes and should be released in the next major version.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@debadree25 @nodejs-github-bot @VoltrexKeyva @mcollina @ShogunPanda @lpinca @JungMinu @trivikr @RafaelGSS