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

Improve examples about usage of runAsyncMain #42

Open
PatrickLehnerXI opened this issue Nov 19, 2021 · 1 comment
Open

Improve examples about usage of runAsyncMain #42

PatrickLehnerXI opened this issue Nov 19, 2021 · 1 comment

Comments

@PatrickLehnerXI
Copy link
Contributor

Some of the examples don't currently use the runAsyncMain function, while others do. This was a bit confusing at first, and doesn't clearly show how DevCmd is intended to be used.

Note: This feedback was provided internally at XITASO, so I'm creating this issue for tracking.

@PatrickLehnerXI
Copy link
Contributor Author

DevCmd can be used with or without using the runAsyncMain, but for our common use case, I do think we should recommend using runAsyncMain.

We should maybe add this to our documentation, and we should definitely make our examples consistent.

However: let's also think about how we can keep integration test coverage for commands that don't use runAsyncMain

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

No branches or pull requests

1 participant