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 actual argument type to error message #3340

Merged
merged 5 commits into from
Aug 13, 2024

Conversation

jtran
Copy link
Contributor

@jtran jtran commented Aug 9, 2024

Related #3338.

Before, we only showed the expected type, not the actual type.

Copy link

qa-wolf bot commented Aug 9, 2024

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

Copy link

vercel bot commented Aug 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
modeling-app ✅ Ready (Inspect) Visit Preview Aug 13, 2024 7:46pm

@jtran jtran force-pushed the jtran/improve-arg-error-message branch from e66f3ce to 2931294 Compare August 9, 2024 20:31
@jtran jtran marked this pull request as ready for review August 9, 2024 20:33
@jtran jtran force-pushed the jtran/improve-arg-error-message branch from 97024c3 to 31f34df Compare August 10, 2024 01:42
@jtran jtran force-pushed the jtran/improve-arg-error-message branch from 31f34df to 3751fe0 Compare August 12, 2024 06:32
@jtran jtran force-pushed the jtran/improve-arg-error-message branch from 3751fe0 to 4c62326 Compare August 12, 2024 14:09
@jtran jtran force-pushed the jtran/improve-arg-error-message branch from 4c62326 to 74d5ed6 Compare August 12, 2024 17:49
@jtran jtran force-pushed the jtran/improve-arg-error-message branch from 5241f75 to d68c11e Compare August 13, 2024 00:08
@jtran jtran force-pushed the jtran/improve-arg-error-message branch from d68c11e to 8f32150 Compare August 13, 2024 00:26
Signed-off-by: Jess Frazelle <github@jessfraz.com>
Signed-off-by: Jess Frazelle <github@jessfraz.com>
@jessfraz jessfraz enabled auto-merge (squash) August 13, 2024 19:58
@jessfraz
Copy link
Contributor

turning on auto merge rust tests all good locally, just engines are getting saturated so might be flakey

@jessfraz jessfraz disabled auto-merge August 13, 2024 20:25
@jessfraz jessfraz merged commit bf9d88e into main Aug 13, 2024
25 of 27 checks passed
@jessfraz jessfraz deleted the jtran/improve-arg-error-message branch August 13, 2024 20:25
@Irev-Dev Irev-Dev mentioned this pull request Aug 14, 2024
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.

2 participants