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

Temporarily override reaonly_attribute? method when reify #1500

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lucy-lufei
Copy link

@lucy-lufei lucy-lufei commented Nov 15, 2024

Implements option 1 in this discussion

Check the following boxes:

  • Wrote good commit messages.
  • Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • Added tests.
  • Added an entry to the Changelog if the new
    code introduces user-observable changes.
  • The PR relates to only one subject with a clear title
    and description in grammatically correct, complete sentences.

@lucy-lufei lucy-lufei force-pushed the ll/override-readonly-attrs-in-reify branch 3 times, most recently from 21b7f1b to a54999a Compare November 15, 2024 23:46
@lucy-lufei lucy-lufei force-pushed the ll/override-readonly-attrs-in-reify branch from a54999a to 160c444 Compare November 15, 2024 23:51
@lucy-lufei lucy-lufei changed the title Temporarily set reaonly attributes to empty when reify Temporarily overrid reaonly_attribute? method when reify Nov 15, 2024
@lucy-lufei lucy-lufei changed the title Temporarily overrid reaonly_attribute? method when reify Temporarily override reaonly_attribute? method when reify Nov 15, 2024
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.

1 participant