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

chore: setup vitest #64

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

chore: setup vitest #64

wants to merge 5 commits into from

Conversation

peetzweg
Copy link
Collaborator

@peetzweg peetzweg commented Feb 6, 2024

As this project is intended as a library to be used by third parties it should work flawlessly on every release and in every edge case. In order to assure this we should add test along the way.

I would suggest using vitest for this as it seem to work great for other projects in the space and we can get inspired it. No need to reinvent the wheel all the time.

I'm not adding excessive tests here as while I wanted to add some test it became clear that we probably need to adapt the usage and api of this library a bit to be better testable.

Copy link

changeset-bot bot commented Feb 6, 2024

🦋 Changeset detected

Latest commit: cdcc37d

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@peetzweg peetzweg changed the title feat: setup vitest chore: setup vitest Feb 6, 2024
@peetzweg peetzweg requested a review from wottpal February 6, 2024 10:12
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.

1 participant