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

Merge main into live #37663

Merged
merged 9 commits into from
Oct 21, 2023
Merged

Merge main into live #37663

merged 9 commits into from
Oct 21, 2023

Conversation

IEvangelist and others added 9 commits October 24, 2022 11:41
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.
@dotnet-bot dotnet-bot added this to the October 2023 milestone Oct 21, 2023
@BillWagner BillWagner merged commit b333c14 into live Oct 21, 2023
23 of 24 checks passed
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

Successfully merging this pull request may close these issues.

6 participants