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

Fix tests, update holochain client, alter build output #118

Closed
wants to merge 16 commits into from

Conversation

mattyg
Copy link
Contributor

@mattyg mattyg commented Oct 24, 2023

I was going to merge the 3 prior prs below into one, but ended up just merging the 2 drafts that I kept open.

I switched back to using webpack and modified its config to output an es module instead of cjs.

Resolves #115

  • replace vite with vitest
  • fix test mocking
  • fix failing tests
  • bump holochain client to 0.16.3 (and remove node 16.x CI support)
  • bump webpack deps
  • switch to es module output, instead of commonjs

@mattyg
Copy link
Contributor Author

mattyg commented Oct 24, 2023

okay this is actually ready @robbiecarlton

@robbiecarlton
Copy link
Contributor

Thanks for this @mattyg - awesome work!

I'm closing this PR as I had to create a new branch to resolve merge conflicts

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.

2 participants