Skip to content
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

v7 alpha: unify keyword code generation [WIP] #1260

Merged
merged 128 commits into from
Sep 1, 2020
Merged

v7 alpha: unify keyword code generation [WIP] #1260

merged 128 commits into from
Sep 1, 2020

Conversation

epoberezkin
Copy link
Member

No description provided.

@epoberezkin epoberezkin changed the base branch from master to v7-alpha August 9, 2020 17:39
@epoberezkin epoberezkin force-pushed the v7-alpha-unify-keywords branch 2 times, most recently from b3e6af8 to f745eca Compare August 9, 2020 18:59
@epoberezkin epoberezkin changed the title v7 alpha: unify standard and custom keyword definition formats v7 alpha: unify standard and custom keyword definition formats [WIP] Aug 10, 2020
@epoberezkin epoberezkin mentioned this pull request Aug 10, 2020
Closed
epoberezkin and others added 12 commits August 30, 2020 08:17
@epoberezkin
refactor: safe code generation
1445d21
@epoberezkin
feat: Code append
e3dd0b9
@epoberezkin
improve codegen performance
694981c
@epoberezkin
refactor: data type checks
a6396b2
@epoberezkin
make Code constructor private, check Name is valid identifier, rename…
e7d72e9 
… Expr enum to Type
@epoberezkin
feat: codegen forOf and forIn loops
e5e846b
@epoberezkin
feat: optional es5 code generation
d1644e6
@epoberezkin
feat: optional newlines in generated code
b1befd9
@epoberezkin
feat: support schemaType in all keywords, including $data; report $da…
6210e4a 
…ta error if $data schema validation fails
@epoberezkin
refactor: unify $data validation for all keywords; "const" and "multi…
6939f90 
…pleOf" valid with undefined
@epoberezkin
browser bundle script (6 tests fail in browser), do not browserify co…
888029f 
…de in schema tests
@epoberezkin
feat: "implements" in keyword definition reserves keyword names (e.g.…
606945c 
…, used by this definition)
@epoberezkin epoberezkin marked this pull request as ready for review August 31, 2020 21:09
@epoberezkin epoberezkin merged commit 700b1d4 into v7-alpha Sep 1, 2020
@epoberezkin epoberezkin deleted the v7-alpha-unify-keywords branch September 1, 2020 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@epoberezkin