-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding publicAPI, bannedApi, and Async analyzers #136
Conversation
@@ -115,7 +115,7 @@ namespace Microsoft.Identity.Abstractions | |||
/// </code> | |||
/// </example> | |||
<# if (framework != "net8"){ #> | |||
#if NET6_0_OR_GREATER | |||
#if NET8_0_OR_GREATER |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't build for NET6.0 any longer.
Updating GiHub action Signed-off-by: Jean-Marc Prieur <jmprieur@microsoft.com>
Directory.Build.props
Outdated
@@ -28,9 +28,17 @@ | |||
<LangVersion>12</LangVersion> | |||
<EnablePackageValidation>true</EnablePackageValidation> | |||
<PackageValidationBaselineVersion>5.0.0</PackageValidationBaselineVersion> | |||
</PropertyGroup> | |||
<BannedApiAnalyzersVersion>3.3.4</BannedApiAnalyzersVersion> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alignment is off
Directory.Build.props
Outdated
@@ -39,7 +47,8 @@ | |||
<!-- The MSAL.snk has both private and public keys --> | |||
<DelaySign>false</DelaySign> | |||
<EnableNETAnalyzers>true</EnableNETAnalyzers> | |||
<EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild> | |||
<EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can simplify a bit if we don't want to have opt outs. These can go, and the conditions in the item groups can go.
</PropertyGroup> | ||
<ItemGroup Label="Build Tools" | ||
Condition="$([MSBuild]::IsOsPlatform('Windows'))"> | ||
<PackageReference Include="Microsoft.SourceLink.GitHub" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI: https://learn.microsoft.com/en-us/dotnet/core/compatibility/sdk/8.0/source-link
sourcelink is in the SDK in net 8 or greater
Adding publicAPI, bannedApi, and Async analyzers
Before starting changes in the public API for the 1P extensibility