-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update Deprecations #1927
Update Deprecations #1927
Conversation
@@ -241,7 +241,7 @@ def _create_event_chain( | |||
feature_name="EventChain", | |||
reason="to avoid confusion, only use yield API", | |||
deprecation_version="0.2.8", | |||
removal_version="0.2.9", | |||
removal_version="0.3.1", |
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.
pushing the deprecation dates back as it will require additional testing
@@ -537,7 +537,7 @@ def call_event_handler( | |||
feature_name="EVENT_ARG API for triggers", | |||
reason="Replaced by new API using lambda allow arbitrary number of args", | |||
deprecation_version="0.2.8", | |||
removal_version="0.2.9", | |||
removal_version="0.3.1", |
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.
pushing this back for the same reason
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.
Seems good 👍
update Deprecation versions and pushed the deprecation versions of deprecated features that require extra testing back