Skip to content

Allow codefixer to annotate accessor declarations as methods#3017

Merged
tlakollo merged 2 commits intodotnet:mainfrom
tlakollo:AllowFixesInAccessors
Sep 1, 2022
Merged

Allow codefixer to annotate accessor declarations as methods#3017
tlakollo merged 2 commits intodotnet:mainfrom
tlakollo:AllowFixesInAccessors

Conversation

@tlakollo
Copy link
Contributor

@tlakollo tlakollo commented Sep 1, 2022

No description provided.

@tlakollo tlakollo added this to the .NET 8.0 milestone Sep 1, 2022
@tlakollo tlakollo self-assigned this Sep 1, 2022
Add more tests
Copy link
Member

@sbomer sbomer left a comment

Choose a reason for hiding this comment

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

Nice!

@tlakollo tlakollo enabled auto-merge (squash) September 1, 2022 22:52
@tlakollo tlakollo merged commit f0b2510 into dotnet:main Sep 1, 2022
@tlakollo tlakollo deleted the AllowFixesInAccessors branch September 2, 2022 00:13
agocke pushed a commit to dotnet/runtime that referenced this pull request Nov 16, 2022
…linker#3017)

Allow codefixer to insert a fix in accessor declarations if we target methods
Fix tests that had accessors and were annotating properties/events
Add tests

Commit migrated from dotnet/linker@f0b2510
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants