Skip to content

Add Vue Testing Library types from DefinitelyTyped #87

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

Merged
merged 4 commits into from
Aug 18, 2019
Merged

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add typescript definitions to DefinitelyTyped
3 participants