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

Remove monorepo structure and tooling #226

Merged
merged 4 commits into from
Aug 16, 2024
Merged

Remove monorepo structure and tooling #226

merged 4 commits into from
Aug 16, 2024

Conversation

hsubox76
Copy link
Collaborator

@hsubox76 hsubox76 commented Aug 14, 2024

When this repo was originally created, we expected to have to create 2 npm packages for web and node. This never materialized so I am removing the unnecessary monorepo structure and tooling. This puts the CHANGELOG at the top level and removes the duplicate READMES, making the repo less complicated to read.

  • Move contents of packages/main to top level.
  • Use top level package.json as json of the actual published package instead of maintaining an umbrella package.json that does not represent a real NPM package
  • Remove lerna and its config
  • Update paths

@hsubox76 hsubox76 marked this pull request as ready for review August 14, 2024 22:15
package.json Outdated Show resolved Hide resolved
samples/.eslintrc.js Outdated Show resolved Hide resolved
scripts/check-format.ts Show resolved Hide resolved
DellaBitta
DellaBitta previously approved these changes Aug 15, 2024
Copy link
Collaborator

@DellaBitta DellaBitta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice finds, @dlarocque .

@DellaBitta
Copy link
Collaborator

Complexity reduction, nice work! 👍

@hsubox76 hsubox76 merged commit 0d4fe06 into main Aug 16, 2024
8 checks passed
@hsubox76 hsubox76 deleted the ch-rm-monorepo branch August 16, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants