Skip to content

Conversation

yardenshoham
Copy link
Contributor

What:

A typo is being corrected.

Why:

A typo should not exist in the comments.

How:

I fixed the typo and corrected the spelling.
Line 45: Asymettic matchers --> Asymmetric matchers

Checklist:

  • Documentation N/A
  • Tests N/A
  • Updated Type Definitions N/A
  • Ready to be merged

Line 45: Asymettic matchers --> Asymmetric matchers
@alexkrolick alexkrolick changed the title fix: typo in to-have-attribute.js chore: typo in to-have-attribute.js comment Sep 18, 2019
@gnapse gnapse merged commit ff238b5 into testing-library:master Sep 19, 2019
@yardenshoham
Copy link
Contributor Author

Thanks for approving my contribution. Could I be added to the contributors so I'll appear in README.md?

@gnapse
Copy link
Member

gnapse commented Sep 19, 2019

Could I be added to the contributors so I'll appear in README.md?

Of course! I thought you weren't interested as this is something you could and should've added in your PR. I would've sworn that we had a checkbox for that in the PR template checklist so as to remind people. Would you mind contributing with that too?

@yardenshoham
Copy link
Contributor Author

OK. On it.

@yardenshoham
Copy link
Contributor Author

It's done. #131.

@gnapse
Copy link
Member

gnapse commented Sep 19, 2019

@all-contributors please add @yardenshoham for docs

@allcontributors
Copy link
Contributor

@gnapse

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

@gnapse
Copy link
Member

gnapse commented Oct 7, 2019

🎉 This PR is included in version 4.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@gnapse gnapse added the released label Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants