-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
change *compiler api #2179
change *compiler api #2179
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
applied @jsumners suggestion 馃憤 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
We must add a very clear call out to this in the release notes.
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
I would like to change the new API interface for *compilers, this would ease the migration from v2 to v3.
I thought it was a good idea to put the
schema
at the end... 馃槄Edit:
the target is that the migration would be: "replace
setSchemaCompiler()
tosetValidatorCompiler()
"cc #2140
Checklist
npm run test
andnpm run benchmark