chore: migrate call sites off the deprecated feature aliases (tokio-runtime, embassy-runtime, critical-section-std-impl)

Đang mở
#458 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
55/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust

Hướng nghiên cứu

Start by checking each listed Makefile, Cargo.toml manifest, and example call site to identify which crate receives each feature; the issue warns that the same names are valid features on the adapter crates. Replace deprecated connector aliases with the canonical features and check for remaining in-tree uses. Then decide whether aliases should be removed now or retained for one release cycle with a CHANGELOG note.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

enhancement

Problem

The connector crates keep feature aliases that their own docs call deprecated:

  • aimdb-mqtt-connector/src/lib.rs:21 — "tokio-runtime: deprecated alias for std"
  • aimdb-knx-connector/src/lib.rs:22-23 — "tokio-runtime and embassy-runtime are deprecated aliases for std and connector respectively, and critical-section-std-impl a deprecated …"
  • aimdb-tcp-connector/Cargo.toml:35-36 — tokio-runtime = ["std"], embassy-runtime = ["connector"]
  • aimdb-websocket-connector/Cargo.toml:31-33 — tokio-runtime = ["server"], kept for existing users

But the repository itself is still the main consumer of those aliases, so they cannot be removed:

  • Makefile: e.g. lines 156, 297, 428, 432, 440, 452–454, 476, 502, 521, 655–675 pass tokio-runtime, embassy-runtime or critical-section-std-impl to connector crates.
  • Manifests: aimdb-client/Cargo.toml:44, aimdb-bench/Cargo.toml:103, aimdb-knx-connector/Cargo.toml:74, aimdb-mqtt-connector/Cargo.toml (84, 104–107, 125–133, 209), tools/embedded-check-fixture/Cargo.toml:33.
  • Examples: tokio-mqtt-connector-demo, tokio-knx-connector-demo, remote-access-demo, sync-api-demo, and the embassy demos.

(tokio-runtime on aimdb-tokio-adapter and embassy-runtime on aimdb-embassy-adapter are real features of those crates, not aliases — those uses are fine and out of scope.)

Suggested work

  1. Replace alias uses on the connector crates with the canonical features (std, connector, server) in the Makefile, crate manifests and examples.
  2. Once nothing in-tree uses them, decide on removal: delete in the next breaking release, or keep for one more cycle with a CHANGELOG note.
  3. aimdb-knx-connector's critical-section-std-impl = [] is already an empty feature and can go as soon as its callers are updated.

Notes

Found by grep over manifests, Makefile and workflows at d9c7be0. Each call site needs a check of which crate the feature is being passed to before editing, because the same names are legitimate on the adapter crates.

Ngôn ngữ chính
Rust
Star
101
Fork
20
Merge trung bình
2 giờ 33 phút
Pull request đã merge (30 ngày)
90

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của aimdb-dev/aimdb

Tất cả issue của aimdb-dev/aimdb

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.