fastify / fastify-dx

Archived
903 stars 43 forks source link

feat(tests): Adds initial testing base for Vue package #18

Closed Eckhardt-D closed 2 years ago

Eckhardt-D commented 2 years ago

Checklist

Description

This is an RFC moreso than a request to merge. I wonder if a new branch tests isn't more suitable? I've chosen vitest and c8 for coverage as these way outperform e.g. Jest in my opinion. This is then also a sign of commitment from my side to continue writing test suites (unit & integration), especially for the Vue package right now. 🙌 @galvez What do you think?

galvez commented 2 years ago

Hey, thank you for this! As a first step you might want to target at main and fix conflicts? :)

Eckhardt-D commented 2 years ago

@galvez Yes, I thought so. And so sorry I never got any notifications on your reply, can I continue with this implementation on main. (Vitest). ?