Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Respond to PR Feedback #18

Merged

Conversation

doug-wade
Copy link
Contributor

@FloEdelmann left some great feedback on #15 (review) after the PR was merged. This addresses that feedback.

@doug-wade
Copy link
Contributor Author

I made an example project if you'd like to test this: https://github.com/doug-wade/jest-fail-on-console-client

@ValentinH
Copy link
Owner

Thanks for this fix. I actually just merged a PR that added the missing methods to the documentation so could you please rebase your PR and adjust accordingly ? 🙂

@doug-wade doug-wade force-pushed the document-fail-on-assertion-failure branch from a3a50af to d9d38c8 Compare April 12, 2022 03:11
@ValentinH ValentinH merged commit ba16aaa into ValentinH:main Apr 12, 2022
@ValentinH
Copy link
Owner

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants