Skip to content
This repository has been archived by the owner on Jul 30, 2020. It is now read-only.

Rename asFragment -> asJSON #34

Merged
merged 1 commit into from
Jun 20, 2019

Conversation

mAAdhaTTah
Copy link
Contributor

The library exports asJSON; asFragment is a DOM function.

What:

Fix name of method in return of render.

Why:

The library returns asJSON while the typings have it returning asFragment.

How:

Updated the typings.

Checklist:

  • Documentation added to the
    docs site (N/A)
  • Typescript definitions updated
  • Tests (N/A
  • Ready to be merged

The library exports `asJSON`; `asFragment` is a DOM function.
@mAAdhaTTah mAAdhaTTah mentioned this pull request Jun 19, 2019
7 tasks
@codecov-io
Copy link

codecov-io commented Jun 19, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #34   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          20     20           
  Lines         262    262           
  Branches       65     65           
=====================================
  Hits          262    262

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 4e30276...f6b3c18. Read the comment docs.

@bcarroll22
Copy link
Collaborator

Awesome, thank you for this!

@allcontributors, please add @mAAdhaTTah for code

@allcontributors
Copy link
Contributor

@bcarroll22

I've put up a pull request to add @mAAdhaTTah! 🎉

@bcarroll22 bcarroll22 merged commit bb22a7c into testing-library:master Jun 20, 2019
@bcarroll22
Copy link
Collaborator

🎉 This PR is included in version 4.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants