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

Jump to bottom

fix(assert): Show caller on panic #134

Merged
merged 3 commits into from Sep 6, 2021
Merged

fix(assert): Show caller on panic #134

merged 3 commits into from Sep 6, 2021

Conversation

epage
Copy link
Contributor

@epage epage commented Sep 6, 2021

Thanks to Rust 1.46, we can show the caller on panic, rather than
assert_cmd, as if assert_cmd was written with macros.

Fixes #133

@epage
style(assert): Order items by interest
0106dd8
@epage
perf(assert): Lazily format output
4318788
@epage
fix(assert): Show caller on panic
150d7be 
Thanks to Rust 1.46, we can show the caller on panic, rather than
`assert_cmd`, as if `assert_cmd` was written with macros.

Fixes assert-rs#133
@epage epage merged commit bb8ef0c into assert-rs:master Sep 6, 2021
@epage epage deleted the context branch September 6, 2021 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add track_callers to asserts
1 participant
@epage