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

Change structured event parser generator to use maps instead of alists #8637

Merged

mrmr1993 merged 1 commit into compatible from feature/register-event-map

Apr 14, 2021

Member

mrmr1993 commented Apr 13, 2021

Checklist:

Document code purpose, how to use it
- Mention expected invariants, implicit constraints
Tests were added for the new behavior
- Document test purpose, significance of failures
- Test names should reflect their purpose
All tests pass (CI will check this if you didn't)
Serialized types are in stable-versioned modules
Does this close issues? List them:

Closes #8600


          Use a map for ppx_register_event parser

ec22a8e

mrmr1993 added the ci-build-me label

mrmr1993 requested a review from a team as a code owner

April 13, 2021 21:14

psteckler approved these changes

View reviewed changes

mrmr1993 merged commit 36fa439 into compatible

mrmr1993 deleted the feature/register-event-map branch

April 14, 2021 02:16

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

Labels