-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Switch to elm-codegen for code generation #6
base: master
Are you sure you want to change the base?
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
495ff1a
to
e585c0f
Compare
Ready for review |
Poke |
Sorry for delay here, got lost in the shuffle. A few notes:
...and many more like this. Please fix this up so that any person cloning this repo for the first time and build and run it properly, and if necessary add some documentation if there are required CLI tools that can't be included as package dependencies. |
Ok, I've changed |
Just noticed some issues with compilation, fixing them |
Blocked by mdgriffith/elm-codegen#101 |
Checking the diff I noticed this is missing the aliases. Going to add them now. |
This is a draft waiting for the new version ofelm-codegen
to be published.It's published now.
This fixes #5