Skip to content

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Oct 1, 2021

Moving the Roslyn Analyzers initialization code from Directory.Build.props to Directory.Build.targets caused it to be applied to code imported from xamarin-android-tools. Previously this did not affect that code because it had its own Directory.Build.props file so it did not traverse the file system upwards any further to find the Java.Interop file.

Fixes for Roslyn issues has not been applied to xamarin-android-tools code, so it adds ~150 warnings to the JI build.

The latest XAT has a Directory.Build.targets, so updating to the latest version will prevent JI's Directory.Build.targets from being used, excluding XAT from the Roslyn Analyzers.

Changes:
dotnet/android-tools@d92fc3e...34e98e2

@jpobst jpobst marked this pull request as ready for review October 1, 2021 20:35
@jpobst jpobst force-pushed the external-warnings branch from ab476da to b88e462 Compare October 4, 2021 19:25
@jpobst jpobst changed the title [external] Disable Roslyn Analyzers on external code. [xamarin-android-tools] Bump to version main@34e98e2b Oct 4, 2021
@jonpryor jonpryor merged commit 4277ac9 into main Oct 4, 2021
@jonpryor jonpryor deleted the external-warnings branch October 4, 2021 20:57
@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants