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

Add ability to identify "auto-implemented" properties #34

Merged
merged 2 commits into from Feb 7, 2018

Conversation

carusology
Copy link
Contributor

Closes #33

Using the definition of "auto-implemented" properties found here, and the underlying techniques (with some modifications) identified here, this implementation does the needful. The test cases are pretty exhaustive, so as long as those pass for a given framework, I think we're good.

@carusology carusology self-assigned this Feb 7, 2018
@carusology carusology merged commit 216757a into master Feb 7, 2018
@carusology carusology deleted the auto-implemented-properties branch February 7, 2018 19:49
@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 99.883% when pulling ca94935 on auto-implemented-properties into 8f30893 on master.

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.

None yet

2 participants