Skip to content

Update CodeAnalysis.src.globalconfig#84059

Merged
stephentoub merged 1 commit into
dotnet:mainfrom
stephentoub:caupdate
Apr 15, 2023
Merged

Update CodeAnalysis.src.globalconfig#84059
stephentoub merged 1 commit into
dotnet:mainfrom
stephentoub:caupdate

Conversation

@stephentoub

Copy link
Copy Markdown
Member
  • Update the names of rules to match the current ones defined in NetAnalyzers
  • Add a few more IDE rules, including enabling IDE0280 Use nameof

@stephentoub
stephentoub requested a review from buyaa-n March 29, 2023 03:38
@ghost ghost added the area-Meta label Mar 29, 2023
@ghost ghost assigned stephentoub Mar 29, 2023
@ghost

ghost commented Mar 29, 2023

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/area-meta
See info in area-owners.md if you want to be subscribed.

Issue Details
  • Update the names of rules to match the current ones defined in NetAnalyzers
  • Add a few more IDE rules, including enabling IDE0280 Use nameof
Author: stephentoub
Assignees: -
Labels:

area-Meta

Milestone: -

@buyaa-n buyaa-n left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

FYI there is build failures caused by IDE0280, otherwise LGTM.

- Update the names of rules to match the current ones defined in NetAnalyzers
- Add a few more IDE rules, including enabling `IDE0280 Use nameof`
@stephentoub

Copy link
Copy Markdown
Member Author

Failures are the known wasm issues impacting practically all PRs.

@stephentoub
stephentoub merged commit ba777a8 into dotnet:main Apr 15, 2023
@stephentoub
stephentoub deleted the caupdate branch April 15, 2023 20:59
@ghost ghost locked as resolved and limited conversation to collaborators May 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants