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

Support private set accessor. #3

Merged
merged 1 commit into from
Feb 14, 2016

Conversation

ciniml
Copy link
Contributor

@ciniml ciniml commented Jan 21, 2016

I often use NotifyPropertyChangedGenerator to implement Model classes.
And these Model classes have some properties whose set accessor is private.
Thus, I changed NotifyPropertyChangedGenerator to keep original access modifier of set accessors after generating codes to notify their changes.

@neuecc
Copy link
Owner

neuecc commented Feb 14, 2016

Thank you, great improvements!

neuecc added a commit that referenced this pull request Feb 14, 2016
@neuecc neuecc merged commit 22bf2da into neuecc:master Feb 14, 2016
@neuecc
Copy link
Owner

neuecc commented Feb 14, 2016

I've released 1.1.1.0 to NuGet

@ciniml
Copy link
Contributor Author

ciniml commented Feb 14, 2016

Thanks to merging PR and updating the NuGet package.
I've reinstalled the package and checked the generator works fine with my project.
Now I can speed up my work more easily.

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.

2 participants