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

Property & field initializers is now preserved in generated constructor. #133

Merged
merged 4 commits into from
Feb 3, 2020

Conversation

NorekZ
Copy link
Contributor

@NorekZ NorekZ commented Jan 22, 2020

Solves this issue.

Generated constructor now contains property & field initializers. They are copied from existing constructor according to this description of how C# compiler compiles them.

Behavior is on by default and can be turned off by setting "PreserveInitializers" config to false.

@NorekZ NorekZ requested a review from SimonCropp January 22, 2020 14:17
@SimonCropp
Copy link
Member

can u add some doco to the readme about this feature

@NorekZ
Copy link
Contributor Author

NorekZ commented Jan 27, 2020

@SimonCropp
Doc updated.

@SimonCropp SimonCropp added this to the 2.1.1 milestone Feb 3, 2020
@SimonCropp SimonCropp merged commit d56cc88 into Fody:master Feb 3, 2020
@SimonCropp SimonCropp added the Bug label Feb 3, 2020
@SimonCropp
Copy link
Member

thanks for the PR. This is now deployed. NuGet may take some time to make it available for download.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants