-
Notifications
You must be signed in to change notification settings - Fork 42
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
docs about nesting ServiceLaunchers #14
Labels
area/documentation
Improvements or additions to documentation.
Milestone
Comments
@weissi this is a great idea, we can def make them compose nicely. to your point on top-level we can potentially compose the type hierarchy differently to make this easier and safer |
@tomerd ah you mean a different type for top-level and not? I think I like that. So one would be called |
Closed
Closed
Yeah I quite like the direction :) |
Merged
closed via #34 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Very clearly ServiceLaucher is helpful for the top-level thing in probably
main.swift
and that's why it has builtin support for handling signals & doing backtraces.However, the lifecycle concept is much bigger than what you do in
main.swift
.Consider the following example:
There, lifecycle is also super useful and I think we should add an example like this to the docs. And the API docs need a big fat warning that catching signals & registering backtraces is only for the top-level ServiceLauncher.
The text was updated successfully, but these errors were encountered: