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

fix error context display & update release workflow #190

Merged
merged 7 commits into from
Aug 21, 2023

Conversation

xxchan
Copy link
Member

@xxchan xxchan commented Aug 21, 2023

revert #189. Error context is printed only for {:?}.

For backtrace, according to anyhow's doc,

If using the nightly channel, or stable with features = ["backtrace"], a backtrace is captured and printed with the error if the underlying error type does not already provide its own. In order to see backtraces, they must be enabled through the environment variables described in std::backtrace

So we can either: ship pre-built binary which is built by stable or unset RUST_BACKTRACE to hide backtrace.


Also update release workflow close #182

revert #189. Error context is printed only for `{:?}`.

For backtrace, according to anyhow's doc,

> If using the nightly channel, or stable with features = ["backtrace"], a backtrace is captured and printed with the error if the underlying error type does not already provide its own. In order to see backtraces, they must be enabled through the environment variables described in std::backtrace

So we can either: ship pre-built binary or unset `RUST_BACKTRACE` to hide backtrace.

Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan xxchan force-pushed the xxchan/desperate-butterfly branch 4 times, most recently from e84a925 to 405e21e Compare August 21, 2023 14:42
Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan xxchan requested a review from skyzh August 21, 2023 15:13
Signed-off-by: xxchan <xxchan22f@gmail.com>
Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan
Copy link
Member Author

xxchan commented Aug 21, 2023

Looks good
image

Signed-off-by: xxchan <xxchan22f@gmail.com>
Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan xxchan changed the title fix error context display fix error context display & update release workflow Aug 21, 2023
@xxchan
Copy link
Member Author

xxchan commented Aug 21, 2023

It can be installed by cargo-binstall now

Signed-off-by: xxchan <xxchan22f@gmail.com>
@TennyZhuang TennyZhuang merged commit d8e0d63 into main Aug 21, 2023
4 checks passed
@TennyZhuang TennyZhuang deleted the xxchan/desperate-butterfly branch August 21, 2023 22:57
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.

ci: enhance release workflow to include more targets
2 participants