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

Add support for custom error messages (closes #9, closes #11) #12

Merged
merged 1 commit into from Jan 25, 2022

Conversation

JCB-K
Copy link
Contributor

@JCB-K JCB-K commented Jan 24, 2022

As discussed in #11 , this PR adds support for custom error message. I also changed the default error message and added mockImplementation there, which should close #9.

@ValentinH
Copy link
Owner

Thanks for this. As soon as I retrieve my admin rights (I left the ricardo-ch org) I'll merge this 🙈

@KevinHerklotz
Copy link
Contributor

Let me know if you want me to merge it 😉

@ValentinH
Copy link
Owner

Yes, please go ahead @KevinHerklotz 🙏 ❤️
I'll do the version bump on NPM afterwards.

@KevinHerklotz KevinHerklotz merged commit 4a0d817 into ValentinH:main Jan 25, 2022
@JCB-K
Copy link
Contributor Author

JCB-K commented Jan 25, 2022

@KevinHerklotz could you cut a new version as well? I guess a commit like this needs to be done before @ValentinH can do the NPM release

@JCB-K JCB-K deleted the custom-error-message branch January 25, 2022 15:36
@ValentinH
Copy link
Owner

Yes you are right 🙈
Sorry for the delay, I'll handle this access issue this week hopefully.
Ultimately, this project will be transferred to my GitHub..

@ValentinH
Copy link
Owner

It was finally published as v2.2.0 ! Sorry for the delay 🙈

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.

Add mockImplementation info to warning message.
3 participants