Skip to content
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

fit(TS): Update TypeScript definitions for getQueriesForElement change #40

Merged
merged 1 commit into from
May 17, 2018

Conversation

pbomb
Copy link
Contributor

@pbomb pbomb commented May 17, 2018

What:

Updated the TypeScript definitions to match the change from bindElementToQueries to getQueriesForElement

Why:

The TS definitions are out of date and are showing the wrong/old function being available.

How:

Renamed the file and its exported function

Checklist:

  • Documentation
  • Tests
  • Ready to be merged
  • Added myself to contributors table

I also update the CONTRIBUTING.md file to remove the -s flag to the setup NPM script as the script already adds that flag.

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.

Super!

@kentcdodds kentcdodds merged commit 597ee1c into testing-library:master May 17, 2018
@kentcdodds
Copy link
Member

Whoops, I just realized that I let a typo slip through in the commit message. I'll push a manual release 👌

kentcdodds pushed a commit that referenced this pull request May 17, 2018
There was an issue with a patch release, so this manual-releases.md
change is to release a new patch version.

Reference: #40
@kentcdodds
Copy link
Member

🎉 This PR is included in version 2.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@pbomb
Copy link
Contributor Author

pbomb commented May 17, 2018

Cool, thanks. Did I make the typo?

@pbomb pbomb deleted the pr/udpate-types branch May 17, 2018 21:30
@kentcdodds
Copy link
Member

We both did. The commit said fit and I didn't notice until after clicking the merge button 😅 No big deal though.

@pbomb
Copy link
Contributor Author

pbomb commented May 17, 2018

🤦‍♂️ oops, sorry. don't know how that happened. 🤷‍♂️ thanks for resolving the issue!

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.

None yet

2 participants