Skip to content

Conversation

@jfversluis
Copy link
Member

@jfversluis jfversluis commented Feb 6, 2025

Description of Change

Resolves a TODO in code to replace the Android FormattedStringExtensions.ToSpannableString overload

Contributes to #8989

@jfversluis jfversluis added this to the .NET 10.0-preview2 milestone Feb 6, 2025
Copilot AI review requested due to automatic review settings February 6, 2025 08:24
@jfversluis jfversluis requested a review from a team as a code owner February 6, 2025 08:24
@jfversluis jfversluis requested review from mattleibow and rmarinho and removed request for a team February 6, 2025 08:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • src/Controls/src/Core/PublicAPI/net-android/PublicAPI.Unshipped.txt: Language not supported
Comments suppressed due to low confidence (1)

src/Controls/src/Core/Platform/Android/Extensions/FormattedStringExtensions.cs:29

  • Ensure that the new method ToSpannableStringNewWay is covered by tests in TestCases.Shared.Tests and can execute properly on TestCases.HostApp.
this FormattedString formattedString,

@dotnet-policy-service
Copy link
Contributor

🚨 API change(s) detected @davidbritch FYI

@jfversluis jfversluis merged commit 19dab0b into net10.0 Feb 7, 2025
21 checks passed
@jfversluis jfversluis deleted the replace-ToSpannableString branch February 7, 2025 07:28
@github-actions github-actions bot locked and limited conversation to collaborators Mar 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants