Skip to content

test: enable missing feature for test build #2820

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.

Sign up for GitHub

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

Open
arjan-bal wants to merge 3 commits into
base: master
Choose a base branch
from
+3 −3
Open

test: enable missing feature for test build #2820

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
205e47e
fix other tests
arjan-bal Aug 15, 2026
ed92e65
remove grpc-gcp example from default feature set
arjan-bal Aug 20, 2026
4515a4e
Merge remote-tracking branch 'source/master' into fix-test
arjan-bal Aug 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion examples/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,7 @@ grpc-helloworld = ["dep:grpc", "dep:grpc-protobuf", "dep:protobuf"]
full = ["gcp", "routeguide", "reflection", "autoreload", "health", "grpc-web",

@dfawley dfawley Aug 20, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it OK that full doesn't include a feature? I guess it's also missing "tower" and "h2" already?

But what's the point in having "full" anyway? Should we delete it and move the things to "default" so it isn't misleading?

"tracing", "uds", "streaming", "mock", "json-codec", "compression", "tls",
"tls-rustls", "tls-client-auth", "types", "cancellation", "h2c",
"grpc-routeguide", "grpc-helloworld", "grpc-gcp"]
"grpc-routeguide", "grpc-helloworld"]
default = ["full"]
# Workaround for cargo-udeps bug.
# TODO: Remove once the fix is released: https://github.com/est31/cargo-udeps/pull/338
Expand Down
2 changes: 1 addition & 1 deletion examples/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ Once your credentials are set up, you will need your GCP Project ID, which can
be found on the main dashboard of the Google Cloud Console. With both of these
ready, you can run the example like so:
```bash
$ cargo run --bin grpc-gcp-client -- <project-id>
$ cargo run --bin grpc-gcp-client --features grpc-gcp -- <project-id>

@dfawley dfawley Aug 20, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is still a required feature for the binary in the cargo.toml though? Does that actually mean you have to set it manually or else it won't run?

```

[Application Default Credentials]: https://docs.cloud.google.com/docs/authentication/application-default-credentials
Expand Down
2 changes: 1 addition & 1 deletion tonic-reflection/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ tonic-prost = { version = "0.14.6", path = "../tonic-prost", default-features =

[dev-dependencies]
tokio-stream = {version = "0.1", default-features = false, features = ["net"]}
tonic = { version = "0.14.6", path = "../tonic", default-features = false, features = ["transport"] }
tonic = { version = "0.14.6", path = "../tonic", default-features = false, features = ["transport", "router"] }

[lints]
workspace = true
Expand Down
Loading