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.

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

Use upstream graphql_ppx, upgrade ppxlib (DEVELOP) #11159

Merged
merged 38 commits into from
Jun 7, 2022

Conversation

Firobe
Copy link
Contributor

@Firobe Firobe commented Jun 2, 2022

Companion PR for #11114

@Firobe Firobe added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label Jun 2, 2022
@Firobe Firobe self-assigned this Jun 2, 2022
@Firobe Firobe linked an issue Jun 2, 2022 that may be closed by this pull request
@Firobe Firobe changed the title [WIP] Upgrade graphql ppx develop Upgrade graphql ppx develop Jun 2, 2022
@Firobe Firobe marked this pull request as ready for review June 2, 2022 14:36
@Firobe Firobe requested review from a team, bkase, psteckler, imeckler and mrmr1993 as code owners June 2, 2022 14:36
@Firobe Firobe changed the title Upgrade graphql ppx develop Use upstream graphql_ppx, upgrade ppxlib (DEVELOP) Jun 2, 2022
@Firobe Firobe requested a review from mitschabaude as a code owner June 2, 2022 17:52
@Firobe Firobe force-pushed the upgrade-graphql-ppx_DEVELOP branch from 0b238ae to 49a992d Compare June 2, 2022 18:06
@Firobe Firobe force-pushed the upgrade-graphql-ppx_DEVELOP branch from 49a992d to 2b9c401 Compare June 2, 2022 18:22
@Firobe
Copy link
Contributor Author

Firobe commented Jun 2, 2022

Ready for review. Similar fixes to the companion PR, except additional changes due to the js_of_ocaml upgrade.

@Firobe Firobe merged commit eb4207f into develop Jun 7, 2022
@Firobe Firobe deleted the upgrade-graphql-ppx_DEVELOP branch June 7, 2022 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade graphql_ppx to use ppxlib
5 participants