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

fix(no-debug): ensure callee exists #81

Merged
merged 1 commit into from
Feb 10, 2020
Merged

fix(no-debug): ensure callee exists #81

merged 1 commit into from
Feb 10, 2020

Conversation

marudor
Copy link
Contributor

@marudor marudor commented Feb 10, 2020

The code used in the tests currently results in an exception as callee does not exist.
This ensures it's there.

Copy link
Member

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

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

Thank you for your PR! Didn't think about this guard against nill callee.

@Belco90 Belco90 merged commit ba34a62 into testing-library:master Feb 10, 2020
@Belco90
Copy link
Member

Belco90 commented Feb 10, 2020

@all-contributors please add @marudor for code and test

@allcontributors
Copy link
Contributor

@Belco90

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

@Belco90
Copy link
Member

Belco90 commented Feb 10, 2020

🎉 This PR is included in version 2.2.2 🎉

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

Successfully merging this pull request may close these issues.

None yet

2 participants