-
Notifications
You must be signed in to change notification settings - Fork 9
Split into package and CLI #214
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
base: main
Are you sure you want to change the base?
Split into package and CLI #214
Conversation
- Since the root project is now an Elm package, all code it needs must be in its src folder. (Packages don't support `source-directories`.) - Add package's src folder to the CLI's `elm.json`.
312c443
to
6f6261a
Compare
We're splitting up the Cli.elm file into two pieces, and that means we'll lose history for one of them. So let's preserve the git history of the bigger piece, the code that will now live in OpenApi.BackendTask.
6f6261a
to
722866d
Compare
The previously installed version had a bug where it required you to install Vite yourself for its use.
577c885
to
5afec96
Compare
Sorry for the force-pushes! The first was to fix a bug (I'd used a slightly outdated The second was to preserve git history for The third was to upgrade elm-pages rather than re-install |
Force pushes are fine in branches, no worries 👍 |
This version is factored into logical commits(!).
Unlike the first version of this effort, this one does not use any tricks to make Elm think the local package is a real package; it just uses
source-directories
for that (and is careful about not having anyGen.
modules in the example project, in order to avoid name collisions).CAVEATS: