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.

Sign up for GitHub

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

Jump to bottom

Abstract properties #7184

Merged
merged 3 commits into from
Feb 22, 2016
Merged

Abstract properties #7184

merged 3 commits into from
Feb 22, 2016

Commits on Feb 22, 2016

  1. Add abstract properties and accessors 

    Almost all the infrastructure is in place, so I just allowed abstract
properties+accessors and added an error when abstract accessors do not
have the same abstractness specified.
    @sandersn
    sandersn committed Feb 22, 2016
    Configuration menu
    Copy the full SHA
    48f728e View commit details
    Browse the repository at this point in the history

  2. Add abstract property tests and rebaseline. 

    1. Positive tests.
2. Negative tests.
3. Update error messages.
4. Remove errors from conformance test.
    @sandersn
    sandersn committed Feb 22, 2016
    Configuration menu
    Copy the full SHA
    02fc8b1 View commit details
    Browse the repository at this point in the history

  3. Update error message as requested in PR comments

    @sandersn
    sandersn committed Feb 22, 2016
    Configuration menu
    Copy the full SHA
    b01a050 View commit details
    Browse the repository at this point in the history