-
Notifications
You must be signed in to change notification settings - Fork 781
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
remove exit-future #5183
remove exit-future #5183
Conversation
1687d97
to
6e4d3d3
Compare
as it is non maintained, and replace with async-channel which is already in the repo.
6e4d3d3
to
b29f6e5
Compare
The tests need to be fixed. Seems the update has removed a field. |
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.
The tests need to be corrected for the new version
…move-exit-future
8720a6c
to
4be6084
Compare
@Mergifyio queue |
🛑 The pull request has been removed from the queue
|
…move-exit-future
@Mergifyio queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at 65c4ff0 |
* remove exit-future usage, as it is non maintained, and replace with async-channel which is already in the repo. * Merge branch 'unstable' of https://github.com/sigp/lighthouse into remove-exit-future * Merge branch 'unstable' of https://github.com/sigp/lighthouse into remove-exit-future
Issue Addressed
the last release was 4 years ago and its repo has been archived. We already have
async-channel
in the repo, so use it instead