Skip to content

Conversation

@Marc-Antoine-Soucy
Copy link
Contributor

GitHub Issue: #

Proposed Changes

  • Bug fix
  • Feature
  • Code style update (formatting)
  • Refactoring (no functional changes, no api changes)
  • Build or CI related changes
  • Documentation content changes
  • Other, please describe:

Description

Updated app to .NET 10, and dropped support for xamarin android, xamarin iOS, and UWP targets.

Impact on version

  • Major (Public API was modified.)
    • Public constructs (class, struct, delegate, enum, etc.) were removed or renamed.
    • Public members were removed or renamed.
    • Public method signatures were changed or renamed.
  • Minor (Public API was extended.)
    • Public constructs, members, or overloads were added.
  • Patch (Public API was unchanged.)
    • A bug in behavior was fixed.
    • Documentation was changed.
  • None (The library is unchanged.)
    • Only code under the build folder was changed.
    • Only code under the .github folder was changed.

PR Checklist

  • Your conventional commits are aligned with the Impact on version section.
  • Documentation is up to date.
    • Content of README.md is up to date.
    • XML documentation is up to date.
  • The BREAKING_CHANGES.md document is up to date.
    • Create a new Major.0.0 header if you do a Major change and list the breaking changes.
  • Tested on all relevant platforms

Other information

Internal Issue (If applicable):

Copy link

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.

Pull request overview

This pull request upgrades the BiometryService library from .NET 7 to .NET 10 and removes support for legacy platforms (Xamarin Android, Xamarin iOS, and UWP). This is a breaking change that modernizes the codebase and aligns with current .NET ecosystem standards.

Changes:

  • Updated target frameworks from net7.0 to net10.0 across all projects
  • Removed support for Xamarin Android, Xamarin iOS, and UWP platforms
  • Updated package dependencies to versions compatible with .NET 10
  • Removed legacy Xamarin sample projects (iOS, Android, UWP)
  • Updated C# language version from 11.0 to 14.0

Reviewed changes

Copilot reviewed 34 out of 53 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/BiometryService/BiometryService.csproj Updated to .NET 10 with modern SDK and removed legacy Xamarin/UWP targets
src/BiometryService/BiometryService.UWP.cs Deleted - UWP implementation removed
src/BiometryService.Abstractions/BiometryService.Abstractions.csproj Updated language version and package references
samples/BiometryService.SampleApp.Uno.Mobile/BiometryService.SampleApp.Uno.Mobile.csproj Updated to .NET 10 with modern Uno platform packages
samples/BiometryService.SampleApp.Uno.WinUI/BiometryService.SampleApp.Uno.WinUI.csproj Updated target framework to .NET 10 Windows
samples/BiometryService.SampleApp.Uno.Shared/App.xaml.cs Simplified application initialization by removing legacy code
samples/BiometryService.SampleApp.Uno.Shared/MainPage.xaml.cs Removed logger factory parameters from BiometryService constructors
samples/BiometryService.SampleApp.Uno.iOS/* Deleted - legacy Xamarin iOS project removed
samples/BiometryService.SampleApp.Uno.Droid/* Deleted - legacy Xamarin Android project removed
samples/BiometryService.SampleApp.Uno.UWP/* Deleted - UWP project removed
BiometryService.sln Removed references to deleted legacy projects
build/azure-pipelines.yml Updated to Windows 2025 agent image
Directory.Build.targets Deleted - SourceLink workaround no longer needed
BREAKING_CHANGES.md Documented breaking changes for version 3.0.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Marc-Antoine-Soucy Marc-Antoine-Soucy merged commit 7faabbc into main Jan 20, 2026
3 checks passed
@Marc-Antoine-Soucy Marc-Antoine-Soucy deleted the dev/maso/net10 branch January 20, 2026 17:47
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.

4 participants