Skip to content

fix: add missing prettyDOM export to typings file #109

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

Conversation

icfantv
Copy link
Contributor

@icfantv icfantv commented Jun 8, 2018

  • exports prettyDOM from dom-testing-library in the
    react-testing-library's typings file.

Fixes: #108

What: adds an export for prettyDOM in react-testing-library's typing's file.

Why: to prevent the Typescript compiler from erroring.

How: an export line was added to the react-testing-library's typing's file.

Checklist:

  • Documentation - N/A
  • Tests - N/A
  • Ready to be merged
  • Added myself to contributors table

* exports `prettyDOM` from dom-testing-library in the
react-testing-library's typings file.
@icfantv
Copy link
Contributor Author

icfantv commented Jun 8, 2018

Not sure what's up with all those diffs in the README...

@gnapse
Copy link
Member

gnapse commented Jun 8, 2018

I think it may have to do with prettier or some other sort of markdown auto formatter. Recently I noticed another PR that changed all the bullet lists in markdown from using - to * as the bullet points, and now your PR is reverting that back.

Copy link
Member

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

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

This is good. It looks like that's what prettier does now

@kentcdodds kentcdodds merged commit 1f953e4 into testing-library:master Jun 8, 2018
@kentcdodds
Copy link
Member

🎉 This PR is included in version 3.1.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@icfantv icfantv deleted the add-missing-prettyDOM-export-to-typings branch June 8, 2018 23:10
julienw pushed a commit to julienw/react-testing-library that referenced this pull request Dec 20, 2018
@nickserv nickserv mentioned this pull request Aug 8, 2023
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.

prettyDOM is not exported from typings file
3 participants