Skip to content
This repository has been archived by the owner on Jun 20, 2018. It is now read-only.

CHE-9836 add output channel API #32

Merged
merged 1 commit into from
Jun 19, 2018
Merged

CHE-9836 add output channel API #32

merged 1 commit into from
Jun 19, 2018

Conversation

olexii4
Copy link

@olexii4 olexii4 commented Jun 18, 2018

Define output channel API for plugin model.
Related issue: eclipse-che/che#9836
Signed-off-by: Oleksii Orel oorel@redhat.com

@olexii4 olexii4 requested review from benoitf, evidolob and mmorhun June 18, 2018 13:47

export class OutputChannelImpl implements theia.OutputChannel {

private _proxy: OutputChannelRegistryMain;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to https://github.com/theia-ide/theia/wiki/Coding-Guidelines: Do not use "_" as a prefix for private properties

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch

Signed-off-by: Oleksii Orel <oorel@redhat.com>
@olexii4 olexii4 merged commit b0e3a27 into master Jun 19, 2018
@olexii4 olexii4 deleted the CHE-9836 branch June 19, 2018 08:10
benoitf pushed a commit that referenced this pull request Jun 19, 2018
Signed-off-by: Oleksii Orel <oorel@redhat.com>
benoitf pushed a commit that referenced this pull request Jun 19, 2018
Signed-off-by: Oleksii Orel <oorel@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants