dotnet / dotnet/android

Audit `EnableNETAnalyzers` usage and fix `CA1305` instances in Xamarin.Android.Build.Tasks

Open
#8,109 0 comments 0 reactions 1 assignee Assigned to @jonathanpeppers View on GitHub
Area: dotnet/android Build
Dominant language
C#
Stars
2.1k
Forks
579
Avg merge
1d 21h
Merged PRs (30d)
257

Description

Context: https://github.com/xamarin/xamarin-android/pull/7993
Context: https://github.com/xamarin/xamarin-android/pull/8110

We should audit/fix instances of `CA1305` in the following projects:

* `src/Xamarin.Android.Build.Tasks/Xamarin.Android.Build.Tasks.csproj`
* `src/Xamarin.Android.Tools.Aidl/Xamarin.Android.Tools.Aidl.csproj`
* `src/Xamarin.Android.Tools.JavadocImporter/Xamarin.Android.Tools.JavadocImporter.csproj`

We should also revisit `CA1309` and `CA1310` (as set in [Configuration.props](https://github.com/xamarin/xamarin-android/blob/941e04c22546227c5e1ae0eca9cda30740d14f9e/Configuration.props#L260) to see if we can enable these errors more widely as well.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.