Skip to content

Conversation

@stephentoub
Copy link
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
Copy link

ghost commented Mar 29, 2023

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: -

Copy link
Contributor

@buyaa-n buyaa-n left a comment

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
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