-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Merge main into live #37663
Merged
Merged
Merge main into live #37663
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Hats were missing `h` in Counter description.
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
…list-packageId-feature Revert "Revert "Tool list package id feature doc update""
* Initial bits * Trying to explore the resilience libs * Updates to resiliency * More updates * Updates to docs * More fixes and updates * Fix heading * More HTTP bits * Apply suggestions from code review Co-authored-by: Martin Taillefer <geeknoid@users.noreply.github.com> * More tweaks * Added image * Additions * More updates * Apply suggestions from code review Co-authored-by: Martin Costello <martin@martincostello.com> Co-authored-by: martintmk <103487740+martintmk@users.noreply.github.com> * Clean up and updates * Add context * Retitle a bit * A quick fix * Resilience in TOC * Updates to headings * Fix TOC entry * Added a bit more detail * Update diagram * Updates to HTTP resiliency content * Add more configuration bits * From peer feedback * Clean up * Apply suggestions from code review Co-authored-by: Martin Costello <martin@martincostello.com> * Update image * Apply suggestions from code review Co-authored-by: martintmk <103487740+martintmk@users.noreply.github.com> * Updates from peer review * Major updates to the resilience bits * A bit closer * The id's should work * There we go * code bits for resilience * Update source with markers * Apply suggestions from code review Co-authored-by: martintmk <103487740+martintmk@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com> Co-authored-by: martintmk <103487740+martintmk@users.noreply.github.com> Co-authored-by: Martin Costello <martin@martincostello.com> * Address all feedback * More clean up, final touches --------- Co-authored-by: Martin Taillefer <geeknoid@users.noreply.github.com> Co-authored-by: Martin Costello <martin@martincostello.com> Co-authored-by: martintmk <103487740+martintmk@users.noreply.github.com> Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
* fix regressions in fundamentals Some of the code structure had changed for newer C# features, and as such, didn't work in Try.NET interactive. In other cases, the snippet missed enough context to be a valid sample that compiled and ran. Run through all the get started and fundamentals samples, make sure all work correctly and run correctly. * remove extra files.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please don't squash-merge this PR.
Internal previews