-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
refactor: remove deprecated EventManager
method addGlobalEventListener
#49645
refactor: remove deprecated EventManager
method addGlobalEventListener
#49645
Conversation
ff2bac7
to
c357532
Compare
Yay! 🎉 |
5e11015
to
8bd31ee
Compare
…ener` This commit removed the deprecated `EventManager` method `addGlobalEventListener`. BREAKING CHANGE: Deprecated `EventManager` method `addGlobalEventListener` has been removed as it is not used by Ivy.
8bd31ee
to
21492fc
Compare
EventManager
method `addGlobalEventList…EventManager
method addGlobalEventListener
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.
Thnx for the cleanup, I had it on one of my TODO lists, happy to see it removed!
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
Reviewed-for: public-api
Reviewed-for: fw-core
Reviewed-for: fw-platform-server
Haha no problem. |
|
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.
reviewed-for: public-api
This PR was merged into the repository by commit 2703fd6. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This commit removes the deprecated
EventManager
methodaddGlobalEventListener
.BREAKING CHANGE: Deprecated
EventManager
methodaddGlobalEventListener
has been removed as it is not used by Ivy.