teamwalnut / graphql-ppx

GraphQL language primitives for ReScript/ReasonML written in ReasonML
https://graphql-ppx.com
MIT License
258 stars 53 forks source link

Ast mapper version is not compatible with bs-platform v7 #210

Closed Sydney-o9 closed 3 years ago

Sydney-o9 commented 3 years ago

@baransu @anmonteiro I've tried to use graphql_ppx with the most recent bs-platform v7 and they are not compatible.

bsb output is:

Failure("Ast_mapper: OCaml version mismatch or malformed input")
  Error while running external preprocessor
Command line: /Users/xxx/Sandbox/reasonml/xxxx-frontend/node_modules/graphql_ppx/ppx '/var/folders/l5/yw1pj_850wl8zb3nyrzkp19w0000gn/T/camlppx664bf5' '/var/folders/l5/yw1pj_850wl8zb3nyrzkp19w0000gn/T/camlppxfde9a1'

package.json is:

  "devDependencies": {
    "bs-platform": "7.3.2"
  },
  "dependencies": {
    "graphql_ppx": "^0.2.8"
  }

This issue seemed to have happened previously with bs-platform v6 - support for Bucklescript 6.x fixed it.

Any ideas if we can get graphql_ppx to work with bs-platform v7?

baransu commented 3 years ago

Correct npm package is "@reasonml-community/graphql-ppx". Please check README how to setup this library 😉

Sydney-o9 commented 3 years ago

Yes! Thank you @baransu :) That fixed it!