Skip to content

chore: Use Sentry.Analyzer#2547

Open
bitsandfoxes wants to merge 1 commit intomainfrom
chore/use-analyzer
Open

chore: Use Sentry.Analyzer#2547
bitsandfoxes wants to merge 1 commit intomainfrom
chore/use-analyzer

Conversation

@bitsandfoxes
Copy link
Contributor

We have it. We added it. We're just not using it.

#skip-changelog

@github-actions
Copy link
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

<PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" Version="1.0.0" PrivateAssets="All" />
<PackageReference Include="Microsoft.CodeAnalysis.BannedApiAnalyzers" Version="3.3.0" PrivateAssets="All" />
<PackageReference Include="Nullable" Version="1.3.0" PrivateAssets="All" />
<ProjectReference Include="$(RepoRoot)src/sentry-dotnet/src/Sentry.Analyzers/Sentry.Analyzers.csproj" OutputItemType="Analyzer" ReferenceOutputAssembly="false" PrivateAssets="all" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: I believe it's common to group PackageReference and ProjectReference in separate ItemGroups ... but this is nitpicking of the highest order

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.

2 participants