Skip to content
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

Add missing required features to Cargo.toml for examples #432

Merged
merged 9 commits into from
Nov 8, 2024

Conversation

ximon18
Copy link
Member

@ximon18 ximon18 commented Nov 1, 2024

Other examples have similar issues. This PR aims to fix them all.

Note to self: I should investigate why tokio-stream needs to be specified explicitly, but for now this at least makes the example easier to run.

@ximon18 ximon18 added bug Something isn't working needs-review labels Nov 1, 2024
@ximon18 ximon18 requested a review from a team November 1, 2024 09:27
@ximon18 ximon18 changed the title Add missing required features to serve-zone example. Add missing required features to Cargo.toml for examples Nov 8, 2024
Copy link
Contributor

@tertsdiepraam tertsdiepraam left a comment

Choose a reason for hiding this comment

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

I confirmed that this checks all the examples correctly. Thanks!

@ximon18 ximon18 merged commit b137d5b into main Nov 8, 2024
34 checks passed
@ximon18 ximon18 deleted the add-missing-serve-zone-required-features branch November 8, 2024 13:16
tertsdiepraam pushed a commit that referenced this pull request Nov 22, 2024
* Add missing required features to serve-zone example.

* Fix required-features listed in Cargo.toml for the query-zone example.

* Fix required-features listed in Cargo.toml for the client-transports example.

* Echo all run commands to check examples, and exit immediately on error (also ensures a final success doesn't cause previous failures to be ignored).

* Fix required-features listed in Cargo.toml for the server-transports example.

* Remove unused import except when feature is enabled that needs it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants