chore(deps): update dependency polyfill to v11#6417
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 72dd486ba1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <PackageVersion Include="OneOf.SourceGenerator" Version="3.0.271" /> | ||
| <PackageVersion Include="Polly" Version="8.7.0" /> | ||
| <PackageVersion Include="Polyfill" Version="10.11.2" /> | ||
| <PackageVersion Include="Polyfill" Version="11.0.0" /> |
There was a problem hiding this comment.
Keep the packaged Polyfill injection in sync
This bump only affects this repo's central package version and the template, but the TUnit.Core NuGet package still packs TUnit.Core.targets, where _TUnitPolyfillVersion remains 10.11.2 (TUnit.Core/TUnit.Core.targets:11). Any .NET Framework/netstandard consumer relying on TUnit's automatic polyfill injection will continue restoring 10.11.2, so the dependency update is not actually applied for that supported path.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Review: Polyfill 10.11.2 → 11.0.0 (Renovate)
This is a minimal, mechanical dependency bump — the version string is updated consistently in both Directory.Packages.props (central package version) and TUnit.Templates/content/TUnit/TestProject.csproj (the net48 template reference). No other code changes. Nothing architectural to flag here since there's no hand-written logic in the diff.
Notes for merging safely:
Polyfillis a compile-time-only source-inlining analyzer package (no runtime dependency shipped), so the blast radius of a bad bump is a build break, not a runtime regression — low risk.- It is still a major version bump (10.x → 11.0.0), and the package was published today with essentially no changelog available yet upstream. Since TUnit multi-targets
net8.0;net9.0;net10.0plus the net48 template, I'd hold off merging until CI (build across all TFMs + snapshot tests) is fully green rather than relying on automerge purely because it's a patch-looking version-string diff. - No previously open review comments on this PR to reconcile against.
Looks good to merge once CI passes. No changes requested.
This PR contains the following updates:
10.11.2→11.0.0Release Notes
SimonCropp/Polyfill (Polyfill)
v11.0.0Compare Source
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate.