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

Refactor into monorepo with compile and cli packages #191

Closed
chrispcampbell opened this issue Jun 4, 2022 · 0 comments · Fixed by #192 or #193
Closed

Refactor into monorepo with compile and cli packages #191

chrispcampbell opened this issue Jun 4, 2022 · 0 comments · Fixed by #192 or #193
Assignees

Comments

@chrispcampbell
Copy link
Contributor

In preparation for adding more packages to this repo, we will:

  • convert this repo to have a monorepo layout (with multiple packages under the packages directory)
  • split the monolithic sdeverywhere package into two new packages:
    • @sdeverywhere/compile
    • @sdeverywhere/cli

The cli package will have new functionality soon, and to prepare for that, we need to separate the core parsing and code gen into its own compile package. There will be a more well-defined interface between the two, and will allow other packages to be incorporated.

As part of this, I'm going to drop the web app generation feature (sde generate --genhtml) and related web sources/dependencies. This will help keep the new cli and compile packages focused on the core functionality. I plan to add some app/library generation features back later once the new build/plugin system falls into place.

@chrispcampbell chrispcampbell self-assigned this Jun 4, 2022
chrispcampbell added a commit that referenced this issue Jun 6, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Fixes #191

BREAKING CHANGE: The `sdeverywhere` package is deprecated and effectively replaced by `@sdeverywhere/cli`.  Additionally, the `sde generate --genhtml` command and supporting code has been removed and will be replaced with a different solution in the near future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant