-
Notifications
You must be signed in to change notification settings - Fork 155
Address feedback in #1670 #1671
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
Conversation
In the cloud-native features section, we should consider removing the orchestration paragraph completely, as it currently reads as though the app host provides orchestration in the cloud rather than being for dev time only. We should consider removing "cloud-native" from the section altogether actually, and focus more on what is actually added, i.e. service discovery and service defaults. |
To clarify, you're advocating for deleting all the Cloud-native content and its subsections? Delete Cloud-native, Observability and telemetry, Health checks, and Resiliency sections? |
Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
Re-frame, rather than delete. No "Cloud-native" heading or positioning. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
Fixes #1670.
Internal previews