-
Notifications
You must be signed in to change notification settings - Fork 599
More consistent readme for msrv #2336
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,6 +2,8 @@ | |
| //! | ||
| //! This library implements a log appender for the [`log`] crate using the [Logs Bridge API]. | ||
| //! | ||
| //! *[Supported Rust Versions](#supported-rust-versions)* | ||
| //! | ||
| //! # Getting Started | ||
| //! | ||
| //! The bridge requires configuration on both the `log` and OpenTelemetry sides. | ||
|
|
@@ -93,6 +95,20 @@ | |
| //! - `with-serde`: Support complex values as attributes without stringifying them. | ||
| //! | ||
| //! [Logs Bridge API]: https://opentelemetry.io/docs/specs/otel/logs/bridge-api/ | ||
| //! | ||
| //! ## Supported Rust Versions | ||
| //! | ||
| //! OpenTelemetry is built against the latest stable release. The minimum | ||
| //! supported version is 1.70. The current OpenTelemetry version is not | ||
| //! guaranteed to build on Rust versions earlier than the minimum supported | ||
| //! version. | ||
| //! | ||
| //! The current stable Rust compiler and the three most recent minor versions | ||
| //! before it will always be supported. For example, if the current stable | ||
| //! compiler version is 1.49, the minimum supported version will not be | ||
| //! increased past 1.46, three minor versions prior. Increasing the minimum | ||
| //! supported compiler version is not considered a semver breaking change as | ||
| //! long as doing so complies with this policy. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should this be in README.md instead of lib.rs?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. that is something I expect to resolve next. My thinking is to make everything in lib.rs and have readme just point to docs.rs/crate |
||
| use log::{Level, Metadata, Record}; | ||
| use opentelemetry::{ | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -32,3 +32,18 @@ management, and export of telemetry. A major goal of OpenTelemetry is that you | |
| can easily instrument your applications or systems, no matter their language, | ||
| infrastructure, or runtime environment. Crucially, the storage and visualization | ||
| of telemetry is intentionally left to other tools. | ||
|
|
||
| *[Supported Rust Versions](#supported-rust-versions)* | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit - do we need this line, given that the information is already clear and easy to find in the README?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. kept it for consistency, and also there'll be more content to be added to this readme in future. |
||
|
|
||
| ## Supported Rust Versions | ||
|
|
||
| OpenTelemetry is built against the latest stable release. The minimum supported | ||
| version is 1.70. The current OpenTelemetry version is not guaranteed to build | ||
| on Rust versions earlier than the minimum supported version. | ||
|
|
||
| The current stable Rust compiler and the three most recent minor versions | ||
| before it will always be supported. For example, if the current stable compiler | ||
| version is 1.49, the minimum supported version will not be increased past 1.46, | ||
| three minor versions prior. Increasing the minimum supported compiler version | ||
| is not considered a semver breaking change as long as doing so complies with | ||
| this policy. | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section needs some changes, the MSRV varies across crates—some require Rust 1.70, while others need 1.71.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point! Let me address it next.