Skip to content
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

Add description to ChannelPluginCompletionStatus #175

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

konifar
Copy link

@konifar konifar commented Jul 9, 2020

Overview

Added description to ChannelPluginCompletionStatus

ChannelPluginCompletionStatus is int enum. If the plugin user wants to show log when Channel IO fails to boot, they have to convert enum to string by themselves.

@konifar
Copy link
Author

konifar commented Jul 10, 2020

@jam0128 @intoxicated Can you review this when you have a time 🙏

@konifar
Copy link
Author

konifar commented Jul 16, 2020

@jam0128 @intoxicated Can you review this? 🙏

@intoxicated
Copy link
Contributor

intoxicated commented Jul 16, 2020

@konifar I think it will break objective-c compatibility :/

@jam0128 jam0128 force-pushed the master branch 2 times, most recently from 4c82595 to 9d10c86 Compare November 20, 2020 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants