Skip to content

Add warning when Authors == AssemblyName#338

Merged
kzu merged 1 commit intomainfrom
dev/DefaultAuthors
Mar 1, 2023
Merged

Add warning when Authors == AssemblyName#338
kzu merged 1 commit intomainfrom
dev/DefaultAuthors

Conversation

@kzu
Copy link
Copy Markdown
Member

@kzu kzu commented Mar 1, 2023

The SDKs set this default in Microsoft.NET.DefaultAssemblyInfo.targets, but it's not really a very useful default (like the Description one).

So we flag this as a warning now.

We take over NG0102 since that wasn't active in any way (analyzers never get 4k chars of a property anyway).

The SDKs set this default in Microsoft.NET.DefaultAssemblyInfo.targets, but it's not really a very useful default (like the Description one).

So we flag this as a warning now.

We take over NG0102 since that wasn't active in any way (analyzers never get 4k chars of a property anyway).
@kzu kzu added the enhancement New feature or request label Mar 1, 2023
@kzu kzu enabled auto-merge (rebase) March 1, 2023 19:04
@kzu kzu merged commit c668502 into main Mar 1, 2023
@kzu kzu deleted the dev/DefaultAuthors branch March 1, 2023 19:16
@devlooped devlooped locked and limited conversation to collaborators Nov 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant