Skip to content

Enable IDE0170 (simplify property pattern) analyzer and fix occurrences#7917

Merged
RussKie merged 1 commit intodotnet:mainfrom
elachlan:IDE0170
Oct 11, 2022
Merged

Enable IDE0170 (simplify property pattern) analyzer and fix occurrences#7917
RussKie merged 1 commit intodotnet:mainfrom
elachlan:IDE0170

Conversation

@elachlan
Copy link
Contributor

@elachlan elachlan commented Oct 11, 2022

Enables IDE0170 analyzer and fix occurrences

Related: #7887

Microsoft Reviewers: Open in CodeFlow

@elachlan elachlan requested a review from a team as a code owner October 11, 2022 06:24
@ghost ghost assigned elachlan Oct 11, 2022
@RussKie
Copy link
Contributor

RussKie commented Oct 11, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Copy link
Contributor

@RussKie RussKie left a comment

Choose a reason for hiding this comment

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

Nice!

@RussKie RussKie enabled auto-merge (squash) October 11, 2022 07:19
@RussKie RussKie added the code cleanup cleanup code for unused apis/properties/comments - no functional changes. label Oct 11, 2022
@elachlan elachlan changed the title Enable IDE0170 analyzer and fix occurrences Enable IDE0170 (simplify property pattern) analyzer and fix occurrences Oct 11, 2022
@elachlan
Copy link
Contributor Author

elachlan commented Oct 11, 2022

Test failures. I will look into it.

Edit: Are they flaky tests?

@dreddy-work
Copy link
Member

Test failures. I will look into it.

Edit: Are they flaky tests?

Seems so. Reset them. Lets see.

@RussKie RussKie merged commit 6e2e029 into dotnet:main Oct 11, 2022
@ghost ghost added this to the 8.0 Preview1 milestone Oct 11, 2022
@elachlan elachlan deleted the IDE0170 branch October 12, 2022 01:48
v-elnovikova pushed a commit to v-elnovikova/winforms that referenced this pull request Oct 18, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Nov 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

code cleanup cleanup code for unused apis/properties/comments - no functional changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants