Skip to content

chore: update Polyfill from 1.32.0 to 9.8.1#4879

Draft
Flash0ver wants to merge 10 commits intomainfrom
chore/update-Polyfill-v9
Draft

chore: update Polyfill from 1.32.0 to 9.8.1#4879
Flash0ver wants to merge 10 commits intomainfrom
chore/update-Polyfill-v9

Conversation

@Flash0ver
Copy link
Member

@Flash0ver Flash0ver commented Jan 28, 2026

Update Polyfill from 1.32.0 to 9.8.1.

Related

Additional Changes

  • remove now redundant Polyfill
  • enable Polyfill-feature FeatureHttp for .NET Standard 2.0 and .NET Standard 2.1
  • ensure Polyfill-feature FeatureMemory is not used when System.Memory is in an incompatible version

@Flash0ver Flash0ver self-assigned this Jan 28, 2026
@Flash0ver
Copy link
Member Author

Flash0ver commented Jan 29, 2026

Potential fixes for making the namespace Polyfills globally available for projects that reference src/Sentry.csproj and has it's internals made been visible to:

  • add <ItemGroup><Using Include="Polyfills" /></ItemGroup> to each of these projects
  • some MSBuild additions and restructuring
  • via an repo-internal Source-Generator utilizing GivesAccessTo

Update:
I went with a simpler conditional <Using Include="Polyfills" ....

@github-actions
Copy link
Contributor

github-actions bot commented Feb 10, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Dependencies ⬆️

  • chore(deps): update Native SDK to v0.12.7 by github-actions in #4920

Other

  • chore: update Polyfill from 1.32.0 to 9.8.1 by Flash0ver in #4879
  • test(android): Use volatile to produce SIGSEGV in native crash test by jpnurmi in #4919

🤖 This preview updates automatically when you update the PR.

@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.84%. Comparing base (9a88d5e) to head (8e1c32d).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4879      +/-   ##
==========================================
- Coverage   73.85%   73.84%   -0.02%     
==========================================
  Files         494      496       +2     
  Lines       17868    17932      +64     
  Branches     3509     3512       +3     
==========================================
+ Hits        13197    13242      +45     
- Misses       3815     3828      +13     
- Partials      856      862       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Flash0ver Flash0ver changed the title chore: update Polyfill from 1.32.0 to 9.7.7 chore: update Polyfill from 1.32.0 to 9.8.1 Feb 16, 2026
Comment on lines -78 to -83
#if !NET7_0_OR_GREATER
public static IReadOnlyDictionary<TKey, TValue> AsReadOnly<TKey, TValue>(this IDictionary<TKey, TValue> dictionary)
where TKey : notnull =>
new ReadOnlyDictionary<TKey, TValue>(dictionary);
#endif

Copy link
Member Author

Choose a reason for hiding this comment

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

note: now included via Polyfill

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.

1 participant