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 example of using TracedError with error kind pattern #635

Merged
merged 1 commit into from
Mar 13, 2020
Merged

Conversation

yaahc
Copy link
Collaborator

@yaahc yaahc commented Mar 13, 2020

The originally given example for how to use TracedError along side the Error Kind pattern was subtly incorrect. You cannot use a derive macro on the error itself because it needs to forward all implementation other than backtrace() to its inner error type, which is not supported by the existing transparent attribute of thiserror.

@hawkw
Copy link
Member

hawkw commented Mar 13, 2020

@yaahc I think if we rebase onto master, CI should start passing again!

Copy link
Member

@hawkw hawkw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! would you mind adding error: to the commit message when merging? beyond that, feel free to merge pending CI.

@yaahc yaahc merged commit 3ec041a into master Mar 13, 2020
@yaahc yaahc deleted the errordoc branch March 13, 2020 21:34
hawkw added a commit that referenced this pull request Oct 23, 2021
# 0.2.0 (October 23, 2021)

This is a breaking change release in order to update the
`tracing-subscriber` dependency version to [the v0.3.x release
series][v03].

### Changed

- Updated `tracing-subscriber` dependency to [v0.3.0][v03] ([#1677])

### Fixed

- Disabled default features of the `tracing` dependency so that
  proc-macro dependencies are not enabled ([#1144])
- Documentation fixes and improvements ([#635], [#695])

### Added

- **SpanTrace**: Added `SpanTrace::new` constructor for constructing a
  `SpanTrace` from a `Span` passed as an argument (rather than capturing
  the current span) ([#1492])

Thanks to @CAD97 for contributing to this release!

[v03]: https://github.com/tokio-rs/tracing/releases/tag/tracing-subscriber-0.3.0
[#635]: #635
[#695]: #695
[#1144]: #1144
[#1492]: #1492
[#1677]: #1677
hawkw added a commit that referenced this pull request Oct 23, 2021
# 0.2.0 (October 23, 2021)

This is a breaking change release in order to update the
`tracing-subscriber` dependency version to [the v0.3.x release
series][v03].

### Changed

- Updated `tracing-subscriber` dependency to [v0.3.0][v03] ([#1677])

### Fixed

- Disabled default features of the `tracing` dependency so that
  proc-macro dependencies are not enabled ([#1144])
- Documentation fixes and improvements ([#635], [#695])

### Added

- **SpanTrace**: Added `SpanTrace::new` constructor for constructing a
  `SpanTrace` from a `Span` passed as an argument (rather than capturing
  the current span) ([#1492])

Thanks to @CAD97 for contributing to this release!

[v03]: https://github.com/tokio-rs/tracing/releases/tag/tracing-subscriber-0.3.0
[#635]: #635
[#695]: #695
[#1144]: #1144
[#1492]: #1492
[#1677]: #1677
kaffarell pushed a commit to kaffarell/tracing that referenced this pull request May 22, 2024
# 0.2.0 (October 23, 2021)

This is a breaking change release in order to update the
`tracing-subscriber` dependency version to [the v0.3.x release
series][v03].

### Changed

- Updated `tracing-subscriber` dependency to [v0.3.0][v03] ([tokio-rs#1677])

### Fixed

- Disabled default features of the `tracing` dependency so that
  proc-macro dependencies are not enabled ([tokio-rs#1144])
- Documentation fixes and improvements ([tokio-rs#635], [tokio-rs#695])

### Added

- **SpanTrace**: Added `SpanTrace::new` constructor for constructing a
  `SpanTrace` from a `Span` passed as an argument (rather than capturing
  the current span) ([tokio-rs#1492])

Thanks to @CAD97 for contributing to this release!

[v03]: https://github.com/tokio-rs/tracing/releases/tag/tracing-subscriber-0.3.0
[tokio-rs#635]: tokio-rs#635
[tokio-rs#695]: tokio-rs#695
[tokio-rs#1144]: tokio-rs#1144
[tokio-rs#1492]: tokio-rs#1492
[tokio-rs#1677]: tokio-rs#1677
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