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

Adds unit tests #5

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Adds unit tests #5

wants to merge 6 commits into from

Conversation

panthershark
Copy link

@panthershark panthershark commented Dec 7, 2022

This adds unit tests.

  • Updates build.js to generate code to a single folder gen, which is nice for separating generated code, tests, and dist.
  • Removes the duplicate package manager. Defaults to npm.
  • Adds prettier configs so the non-default settings (single quote mostly) are automatic for contributors.
  • Removes unused rollup config
  • adds vite and vitest to support unit tests.
  • updates all deve dependencies to latest.

To see the tests fail, git checkout 8308710e6786b92883423ce7f71c6f7987a55e29. This is the commit where the tests are setup, but the fixes are not in place.

To see the tests pass, checkout the tip of this branch.

…ies build.js to emit to a gen folder so tests, dist, and gen code can be separated.
* updates packaging and tests to pass. updates to jsx-runtime over jsx classis

* update peer dependency range since jsx-runtime updates started shipping in preact 10.5.0
@ekwoka
Copy link
Owner

ekwoka commented Dec 18, 2022

I'm trying to find a way to cherry pick the testing stuff from this but it's all mixed together with other reworks.

I solved the issue we discussed with your inspiration but there are some other changes here I don't totally agree with, but the testing would be useful

@ekwoka ekwoka force-pushed the master branch 3 times, most recently from 607a064 to 0ba9178 Compare February 3, 2023 14:56
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