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

enable async dispatcher macros only for tests #193

Merged
merged 2 commits into from
May 31, 2024
Merged

Conversation

rgbkrk
Copy link
Member

@rgbkrk rgbkrk commented May 31, 2024

Part of this is a code cleanup to gate the tests behind cfg(test). The rest is to enable the async dispatcher macros for the tests (defining main and test macros for handling) and ensuring they aren't exposed for downstream use.

@rgbkrk rgbkrk changed the title wrap tests in cfg(test) enable async dispatcher macros only for tests May 31, 2024
@rgbkrk rgbkrk merged commit 01355be into master May 31, 2024
3 checks passed
@rgbkrk rgbkrk deleted the wrap-tests-in-cfg-test branch May 31, 2024 23:38
@rgbkrk
Copy link
Member Author

rgbkrk commented Jun 1, 2024

Going to come back to this one to make test support a more general macro.

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.

1 participant