Make the `sentry` crate's `serde_json` feature a no-op
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 25/100
Hướng nghiên cứu
Start with sentry/Cargo.toml and inspect the [dependencies] and [features] entries for serde_json, while leaving the log feature unchanged. Remove the optional dependency and retain an empty, documented serde_json feature; done means existing users can still enable that feature and cargo machete no longer flags the dependency.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The sentry crate declares an optional dependency on serde_json, but nothing in sentry/src uses it (cargo machete flags it). Because it isn't referenced with dep: in [features], Cargo exposes it as an implicit serde_json feature, and that feature ships in the published 0.49.5 manifest. Deleting the dependency outright would therefore remove a public feature.
Scope:
- Remove the unused optional
serde_jsondependency fromsentry/Cargo.tomlwhile keepingserde_jsonas an empty, soft-deprecated feature, so existingfeatures = ["serde_json"]users keep compiling. - Unrelated to
sentry'slogfeature, which enables thesentry-logintegration and stays as is. - Cargo has no stable way to deprecate a feature in a way that warns downstream users. Build-script
cargo::warningoutput and#[deprecated]lints are hidden for non-path dependencies, and the feature deprecation RFC is still open. So the deprecation can only be documented.
via @szokeasaurusrex.
--
- Ngôn ngữ chính
- Rust
- Star
- 755
- Fork
- 192
- Merge trung bình
- 3 ngày 9 giờ
- Pull request đã merge (30 ngày)
- 13
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của getsentry/sentry-rust
-
breaking change
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
getsentry/sentry-rust#1387 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
breaking change Rust
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/sentry-rust#1386 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
`Transport::flush` on an `Arc<T>` transport returns `true` without flushing `T`Có thể đã có người làm @sentry-junior đã nhận 4 ngày trước. Đang mởBug Rust
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
getsentry/sentry-rust#1374 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
breaking change Rust Task
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
getsentry/sentry-rust#1345 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Pin Rust toolchainĐang mởRust Task
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
getsentry/sentry-rust#933 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của getsentry/sentry-rust
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
libnativeapi/nativeapi#21 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
longbridge/nsh#804 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày