xamarin-fingerprint icon indicating copy to clipboard operation
xamarin-fingerprint copied to clipboard

CA1416 analyzer errors (This call site is reachable on...) for .NET MAUI version

Open jfversluis opened this issue 2 years ago • 5 comments

When installing the 3.0.0-beta1 version on a .NET MAUI project I get CA1416 warnings all over the place. We had that happening as well, so you probably want to have a look at the fixes we did. See: https://github.com/dotnet/maui/issues/823

The app runs regardless, but the warnings might be confusing. Only Android is shown in the screenshot below, but happens on all platforms.

image

Steps to reproduce

  1. Create new .NET MAUI project
  2. Install Plugin.Fingerprint 3.0.0-beta1
  3. Start adding the Plugin.Fingerprint code

Expected behavior

No analyzer warnings

Actual behavior

Analyzer warnings all over the place to inform the user about supported platform versions

Crashlog

N/A

Configuration

Version of the Plugin: 3.0.0-beta1

Platform: Android, iOS, Mac Catalyst, Windows

Device: N/A

jfversluis avatar May 30 '22 10:05 jfversluis

Hi @jfversluis,

I'm the friendly issue checker. Thanks for using the issue template :star2: I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

smsissuechecker avatar May 30 '22 10:05 smsissuechecker

Can anyone help with this issue?

ghost avatar Jul 07 '22 09:07 ghost

Hope this doc would help Handle reported warnings, let us know if you have any more questions

buyaa-n avatar Jul 12 '22 16:07 buyaa-n

Hope this doc would help Handle reported warnings, let us know if you have any more questions

Hi @buyaa-n, The component is requesting at least Android 31.0 and as the Android 31.0 version is a recent version and will disqualify many devices. Will the minimum version for MAIU be Android 31.0?

ghost avatar Oct 11 '22 12:10 ghost

I am not familiar with this repo, so I am not sure if this should be fixed in this repo or MAUI, if this repo is not using anything that require Android 31.0+ this repo should update its minimum version down

buyaa-n avatar Oct 11 '22 16:10 buyaa-n