Skip to content

docs: correct Polly migration guide - #376

Merged
thomhurst merged 2 commits into
mainfrom
issue-340-polly-migration-docs
Aug 27, 2026
Merged

docs: correct Polly migration guide#376
thomhurst merged 2 commits into
mainfrom
issue-340-polly-migration-docs

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Summary

  • correct Polly context, retry-number, synchronous-execution, HTTP hedging, and jitter mappings
  • document standard HTTP and chaos default differences with executable contracts
  • expand Polly v7 mappings for HTTP, circuit breakers, safe execution, rate limiting, and caching
  • compile every added Kevlar example against packed packages

Closes #340.

Validation

  • dotnet build Kevlar.slnx -c Release
  • dotnet pack Kevlar.slnx -c Release --no-build -p:Version=0.0.0-doc340
  • pwsh scripts/Verify-Docs.ps1
  • pwsh scripts/Verify-DocSnippets.ps1 -PackagesPath artifacts/package/release -Version 0.0.0-doc340 -NoImplicitUsings
  • npm ci (from docs/)
  • npm run build (from docs/)

Docusaurus production build passed. A screenshot was not captured because no browser backend was connected in this session.

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 67dd6dd0-37f6-4bda-80c9-b842d24363ed


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thomhurst
thomhurst force-pushed the issue-340-polly-migration-docs branch from ad631bd to fe5f0b1 Compare August 27, 2026 02:47
@thomhurst
thomhurst merged commit b1e0dc4 into main Aug 27, 2026
7 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.

docs(polly-migration): wrong rows, missing standard-HTTP and chaos defaults, thin v7 table

1 participant