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

subscriber: update minimum tracing-core dep to 0.1.28 #2190

Merged
merged 1 commit into from Jun 30, 2022

Conversation

hawkw
Copy link
Member

@hawkw hawkw commented Jun 30, 2022

Motivation

PR #2008 added a new method to the tracing_core::Subscriber trait, and
modified tracing-subscriber to and implement that method. However,
that PR did not increase the minimum tracing-core dependency for the
tracing-subscriber crate.

This means that if a dependent of tracing-subscriber updates only
tracing-subscriber dependency version (e.g. by running
cargo update -p tracing-subscriber), it will not also update its
tracing-core version to one that contains the new method, and
tracing-subscriber will fail to compile. This is a common occurrence
with projects using Dependabot, for example.

Solution

This commit updates tracing-subscriber's minimum tracing-core dep to
0.1.28. Once this merges, I'll release 0.3.13 of tracing-subscriber
and yank 0.3.12.

PR #2008 added a new method to the `tracing_core::Subscriber` trait, and
modified `tracing-subscriber` to and implement that method. However,
that PR did *not* increase the minimum `tracing-core` dependency for the
`tracing-subscriber` crate.

This means that if a dependent of `tracing-subscriber` updates *only*
`tracing-subscriber` dependency version (e.g. by running
`cargo update -p tracing-subscriber`), it will not also update its
`tracing-core` version to one that contains the new method, and
`tracing-subscriber` will fail to compile. This is a common occurence
with projects using Dependabot, for example.

This commit updates `tracing-subscriber`'s minimum `tracing-core` dep to
0.1.28. Once this merges, I'll release 0.3.13 of `tracing-subscriber`
and yank 0.3.12.
@hawkw hawkw requested a review from a team as a code owner June 30, 2022 18:15
@hawkw hawkw enabled auto-merge (squash) June 30, 2022 18:20
@hawkw hawkw merged commit b4680b1 into v0.1.x Jun 30, 2022
@hawkw hawkw deleted the eliza/bump-core-dep branch June 30, 2022 18:27
hawkw added a commit that referenced this pull request Jun 30, 2022
# 0.3.13 (Jun 30, 2022)

This release of `tracing-subscriber` fixes a compilation failure due to
an incorrect `tracing-core` dependency that was introduced in v0.3.12.

### Changed

- **tracing_core**: Updated minimum dependency version to 0.1.28 (#2190)
hawkw added a commit that referenced this pull request Jun 30, 2022
# 0.3.13 (Jun 30, 2022)

This release of `tracing-subscriber` fixes a compilation failure due to
an incorrect `tracing-core` dependency that was introduced in v0.3.12.

### Changed

- **tracing_core**: Updated minimum dependency version to 0.1.28 (#2190)
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.

None yet

2 participants