Skip to content

Conversation

afontcu
Copy link
Member

@afontcu afontcu commented Aug 18, 2019

Closes #84

Following the strategy outlined here testing-library/react-testing-library#437, I'm adding VTL types as a dependency.

This means that people installing @testing-library/vue would get @types/testing-library__vue installed automatically. So from the developers point of view, it's as though the type definitions shipped with the package directly. (adapted from here).

imatge

@afontcu afontcu requested review from alexkrolick and dfcook August 18, 2019 09:03
@afontcu afontcu changed the title Add Vue Testing Library types from Add Vue Testing Library types from DefinitelyTyped Aug 18, 2019
@afontcu afontcu added the enhancement New feature or request label Aug 18, 2019
@codecov
Copy link

codecov bot commented Aug 18, 2019

Codecov Report

Merging #87 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #87 +/- ## ===================================== Coverage 100% 100% ===================================== Files 1 1 Lines 63 63 Branches 10 10 ===================================== Hits 63 63

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 726491b...d0a4495. Read the comment docs.

@testing-library testing-library deleted a comment from codecov bot Aug 18, 2019
Copy link
Collaborator

@dfcook dfcook left a comment

Choose a reason for hiding this comment

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

Great stuff

@afontcu afontcu merged commit 297c1c1 into master Aug 18, 2019
@afontcu afontcu deleted the add-types branch August 18, 2019 20:27
@afontcu
Copy link
Member Author

afontcu commented Aug 18, 2019

🎉 This PR is included in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
3 participants