Skip to content
This repository has been archived by the owner on Dec 8, 2021. It is now read-only.

Add support for ReasonML #253

Open
schickling opened this issue Nov 4, 2018 · 5 comments
Open

Add support for ReasonML #253

schickling opened this issue Nov 4, 2018 · 5 comments

Comments

@schickling
Copy link
Contributor

In the initial version of graphqlgen (formerly graphql-resolver-codegen) there was a first implementation of ReasonML support. We should bring this functionality back to life 🚀

Would be awesome if someone wants to take a stab at this via a PR 🙏

@czystyl
Copy link

czystyl commented Nov 5, 2018

Hi @schickling, currently doing ReasonML in my free time so if you would like to hang out and discuss would be great! You can ping me on Prisma Slack as (czysty)

@Weakky
Copy link
Contributor

Weakky commented Nov 5, 2018

Hey @czystyl !

I suggest you wait a little bit for #244 to be merged before spending too much time implementing ReasonML. #244 implements a simplified AST to abstract most languages (ReasonML should fit well), allowing us to share lots of common logic between the different languages for code generation and scaffolding 🙌

@samuela
Copy link
Contributor

samuela commented Nov 7, 2018

This would be a huge win for me!

@schickling
Copy link
Contributor Author

Now that #244 is merged, it would be great if someone wants to pick this up. A first step will be to propose a file/typing layout similar to #115

@czystyl
Copy link

czystyl commented Nov 16, 2018

I was trying to catch @Weakky on Slack but without result. @schickling can we hang out? It would be awesome to get some information from core team ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants