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

Raise an Error When Computed Field Overrides Field #7346

Merged
merged 4 commits into from Sep 5, 2023
Merged

Raise an Error When Computed Field Overrides Field #7346

merged 4 commits into from Sep 5, 2023
+19 −0

Commits on Sep 5, 2023

  1. throw error on computed field override 

    throw error when computed field overrides field from super class
    @sydney-runkle
    sydney-runkle committed Sep 5, 2023
    Configuration menu
    Copy the full SHA
    e751a76 View commit details
    Browse the repository at this point in the history

  2. raise error on field override, adding needed import and unit test

    @sydney-runkle
    sydney-runkle committed Sep 5, 2023
    Configuration menu
    Copy the full SHA
    b99d117 View commit details
    Browse the repository at this point in the history

  3. Use match pattern for pytest raises context manager 

    Co-authored-by: Adrian Garcia Badaracco <1755071+adriangb@users.noreply.github.com>
    @sydney-runkle @adriangb
    sydney-runkle and adriangb committed Sep 5, 2023
    Configuration menu
    Copy the full SHA
    2aa43d5 View commit details
    Browse the repository at this point in the history

  4. quick quote formatting fix

    @sydney-runkle
    sydney-runkle committed Sep 5, 2023
    Configuration menu
    Copy the full SHA
    9b9a435 View commit details
    Browse the repository at this point in the history