-
Notifications
You must be signed in to change notification settings - Fork 631
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
Adapt to the new EventLoop API for checking the channel compatibility #2367
Conversation
should we ignore the CI checks errors ? |
No I need to check why the UDS tests are failing |
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
ignore my last LGTM, sorry, that was a mistake 😊. Will review again after tests have passed. |
@pderop I created an issue for |
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.
looks good to me.
@pderop Thanks |
Related to #1873 |
Fixes #2291