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

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
Copy link

This PR has been automatically marked as stale due to inactivity.
The resources of our volunteers are limited.
If this is something you are committed to continue working on, please address any concerns raised by review and/or ping us again.
Thank you for all your contributions.

@github-actions github-actions bot added the Stale label Feb 14, 2025
@github-actions github-actions bot closed this Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant