Skip to content

chore(deps): update dependency polyfill to 11.4.1 - #6899

Merged
thomhurst merged 1 commit into
mainfrom
renovate/polyfill-11.x
Sep 27, 2026
Merged

thomhurst merged 1 commit into
mainfrom
renovate/polyfill-11.x

Conversation

@thomhurst

@thomhurst thomhurst commented Sep 27, 2026 •

Copy link
Copy Markdown
Owner

This PR contains the following updates:

Package Type Update Change
Polyfill nuget patch 11.4.0 → 11.4.1

Release Notes

SimonCropp/Polyfill (Polyfill)

v11.4.1

Compare Source

  • #​612 Fix ReadAllLines disposing signal while handler still running

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 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.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate CLI.

@thomhurst thomhurst added dependencies Pull requests that update a dependency file PATCH renovate-bot labels Sep 27, 2026
@thomhurst
thomhurst enabled auto-merge (squash) September 27, 2026 11:36
@thomhurst
thomhurst deployed to Pull Requests September 27, 2026 11:36 — with GitHub Actions Active
@thomhurst
thomhurst deployed to Pull Requests September 27, 2026 11:36 — with GitHub Actions Active
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-27T13:14:05.806742Z 5372dad New commits
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 12ce77ac-a243-4685-99b5-d56def3a30f4

📥 Commits

Reviewing files that changed from the base of the PR and between 67b7ce3 and 5372dad.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 115e089e-05f3-46c5-a6ec-70f40348182e

📥 Commits

Reviewing files that changed from the base of the PR and between 0d44237 and 25cee16.

📒 Files selected for processing (2)
  • Directory.Packages.props
  • src/TUnit.Templates/content/TUnit/TestProject.csproj

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 5 remain after this review.


📝 Walkthrough

Walkthrough

The central Polyfill package version and the version used by the .NET Framework test project template changed from 11.4.0 to 11.4.1.

Changes

Polyfill version update

Layer / File(s) Summary
Update Polyfill versions
Directory.Packages.props, src/TUnit.Templates/content/TUnit/TestProject.csproj
Both the centrally managed package version and the template’s conditional package reference changed to 11.4.1.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 25cee

Both package references are updated to 11.4.1, with no concrete PR-specific issue evident in the supplied changes. No merge blocker remains.

Architecture Summary

Architecture risk: 🔵 Low · up to 25cee

The change affects 2 systems.

Changed systems: Directory.Packages.props, src

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — Directory.Packages.props (service) was modified; 1 changed file maps to changed impact.
  • observed — src (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in Directory.Packages.props: The centrally managed Polyfill version changed from 11.4.0 to 11.4.1.
  • observed — Modified behavior in src/TUnit.Templates/content/TUnit/TestProject.csproj: The Polyfill package version changed from 11.4.0 to 11.4.1 in the IsNetFramework conditional.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: updating the Polyfill dependency to version 11.4.1.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

A rabbit checks the version line,
And finds the new release is fine.
One point up, the packages agree,
The template joins the update spree.
Then hops away beneath the tree.

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

@greptile-apps

greptile-apps Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[High risk] Updates a shared dependency across the project.

The PR appears safe to merge; the previously identified Polyfill version mismatch is corrected.

Summary

The PR updates Polyfill from 11.4.0 to 11.4.1 in central package management and the .NET Framework project template. Since the previous review, it also updates the version automatically referenced by TUnit.Core, aligning all three locations.

Reviews (3) · Last reviewed commit: "chore(deps): update dependency polyfill ..."

Comment thread Directory.Packages.props

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 67b7ce34ea

ℹ️ 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".

Comment thread Directory.Packages.props
<PackageVersion Include="OneOf.SourceGenerator" Version="3.0.271" />
<PackageVersion Include="Polly" Version="8.8.0" />
<PackageVersion Include="Polyfill" Version="11.4.0" />
<PackageVersion Include="Polyfill" Version="11.4.1" />

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Update the auto-injected Polyfill version

When a .NET Framework or netstandard consumer relies on TUnit.Core's default automatic Polyfill injection instead of declaring the package explicitly, the packaged src/TUnit.Core/TUnit.Core.targets still sets _TUnitPolyfillVersion to 11.4.0 and uses it for both Version and VersionOverride. Consequently, those consumers will not receive the 11.4.1 fix despite this dependency update; update that embedded version alongside the central and template references.

Useful? React with 👍 / 👎.

This was referenced Sep 30, 2026

This branch was successfully deployed

1 active deployment
Pull Requests — 5372dad7 Deployed Sep 27, 2026 by thomhurst via modularpipeline (ubuntu-latest) #19532
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file PATCH renovate-bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants