Nothing Special   »   [go: up one dir, main page]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tostring hoisting #688

Merged
merged 21 commits into from
Oct 27, 2024
Merged

Tostring hoisting #688

merged 21 commits into from
Oct 27, 2024

Conversation

SteveDunn
Copy link
Owner

Implements #593

Introduced a test to skip `ToString` methods that implement an interface in the snapshot tests. Also modified the code generation logic to change ToString method generation to use parameters instead of items and added a detailed method documentation.
Cleaned up the GenerateCodeForToString.cs file by eliminating several blocks of commented-out code. This improves readability and maintains a cleaner codebase.
This commit simplifies the code by removing commented-out `createChangedSolution` lines in both `AddMethodCodeFixProvider` and `AddStaticToExistingMethodCodeFixProvider` classes. This helps to clean up the code and ensure that only the necessary logic is present.
@SteveDunn SteveDunn merged commit 7645a45 into main Oct 27, 2024
6 checks passed
@SteveDunn SteveDunn deleted the tostring-hoisting branch October 27, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant