-
-
Notifications
You must be signed in to change notification settings - Fork 106
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
Improve not
error messages
#101
Comments
From #71 (comment)
|
I think we can automate most of it by creating helpers for the most common cases, like
Could be automated to something like: expectToBeOfType('bar', 'string', 'number'); |
Good idea! I'll take a look when I find some time. |
@SamVerschueren @sindresorhus Can you help getting me started here. i want to start my first OS contribution. |
@issuehuntfest has funded $60.00 to this issue. See it on IssueHunt |
AFAIU in almost all cases it's enough to just put |
Any further work being done on this issue? The proposed PR doesn't pass tests. I'm inclined to pick this up. |
@kainiedziela the project's structure has changed since the PR was opened, so it's slightly outdated, I'll revisit the PR and update it tomorrow. |
@sindresorhus has rewarded $54.00 to @alexnoz. See it on IssueHunt
|
Right now we just prefix them with
[NOT]
, but this is not very nice.IssueHunt Summary
alexnoz has been rewarded.
Backers (Total: $60.00)
Submitted pull Requests
not
messagesTips
IssueHunt has been backed by the following sponsors. Become a sponsor
The text was updated successfully, but these errors were encountered: