-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix: Remove optional variable attribute experiment from project #1011
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
03c5660
to
aaca4d9
Compare
@bryantbiggs
|
I think you can remove And also user can use it in their code directly. |
What does this PR do?
< 1.3.0
from the project itself, but there is additional work still before users can use 1.3.0+ on this project. Namely, the two external addons needed to be updatedMotivation
More
pre-commit run -a
with this PRNote: Not all the PRs require a new example and/or doc page. In general:
docs/add-ons/*
is required for new a new addonFor Moderators
Additional Notes