Skip to content

Conversation

@ipetkov
Copy link
Member

@ipetkov ipetkov commented Sep 29, 2025

Should make it a little bit nicer for Nix/devshell users

@ipetkov ipetkov requested a review from a team as a code owner September 29, 2025 16:41
It looks like `tonic_web::enable` was removed with the 0.13.x release so
this code hasn't actually compiled for a long time. The doc and code
examples all use a custom cors layer so whatever heavy lifting the 0.12
method was doing doesn't seem necessary anymore
Features are additive so this should be safe to lint them all together
(and avoid the combinatorics of using cargo-hack)
@ipetkov
Copy link
Member Author

ipetkov commented Sep 29, 2025

Depends on #640 which should fix CI runs

1. We've already committed our lockfile so there's no need to test the
   API/subscriber crates individually (if we want to test with the
   latest available versions we should be running a `cargo update` in
   CI, so this change remains compatible with the previous behavior)
2. Also enable some non-default feature flags to ensure they are being
   tested
Flake lock file updates:

• Updated input 'flake-utils':
    'github:numtide/flake-utils/cfacdce06f30d2b68473a46042957675eebb3401' (2023-04-11)
  → 'github:numtide/flake-utils/11707dc2f618dd54ca8739b309ec4fc024de578b' (2024-11-13)
• Updated input 'nixpkgs':
    'github:NixOS/nixpkgs/f9d39fb9aff0efee4a3d5f4a6d7c17701d38a1d8' (2024-02-11)
  → 'github:NixOS/nixpkgs/e9f00bd893984bc8ce46c895c3bf7cac95331127' (2025-09-28)
• Updated input 'rust-overlay':
    'github:oxalica/rust-overlay/8dfbe2dffc28c1a18a29ffa34d5d0b269622b158' (2024-02-13)
  → 'github:oxalica/rust-overlay/be3b8843a2be2411500f6c052876119485e957a2' (2025-09-29)
• Removed input 'rust-overlay/flake-utils'
@seanmonstar seanmonstar merged commit d3848d7 into tokio-rs:main Sep 29, 2025
14 of 15 checks passed
@ipetkov ipetkov deleted the push-vvysqzvryqyo branch September 29, 2025 18:21
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