Skip to content

Update README.md #191

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

Closed
wants to merge 1 commit into from
Closed

Update README.md #191

wants to merge 1 commit into from

Conversation

shicholas
Copy link

Use new non-deprecated API

Use new non-deprecated API
@@ -77,9 +77,11 @@ struct Application {
let service2 = FooService()

let serviceGroup = ServiceGroup(
services: [service1, service2],
configuration: .init(gracefulShutdownSignals: [.sigterm]),
Copy link
Author

Choose a reason for hiding this comment

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

this line gave me a compiler warning when using the latest. in doc-c the newer docs seem like they're up to date.

@FranzBusch FranzBusch added the semver/none No version bump required. label Dec 11, 2024
@FranzBusch
Copy link
Contributor

Thanks for the contribution. @shicholas Can you resolve the merge conflicts here?

@shicholas
Copy link
Author

it looks like someone fixed this already, which is the merge conflict, so will close this PR, thanks!

@shicholas shicholas closed this Dec 11, 2024
@shicholas shicholas deleted the patch-1 branch December 11, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants