-
Notifications
You must be signed in to change notification settings - Fork 29
Sonatype Release #93
base: master
Are you sure you want to change the base?
Sonatype Release #93
Conversation
key <- channel.register(selector, JSelectionKey.OP_ACCEPT) | ||
buffer <- Buffer.byte(256) | ||
_ <- started.succeed(()) | ||
// key <- channel.register(selector, JSelectionKey.OP_ACCEPT) |
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.
fmt linter got failed on unused ones. so i commented them for now
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.
@ratoshniuk this line is needed for the test to pass. Just change key
to _
and it should work.
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.
@ratoshniuk can you address this comment?
@ratoshniuk pls merge in master, this should solve current issue! |
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.
@ratoshniuk please make requested change.
No description provided.