-
Notifications
You must be signed in to change notification settings - Fork 149
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
Delete deprecated Theia editor plugin version #127
Conversation
Please be careful and do not update prod due to redhat-developer/rh-che#1379 |
@ibuziuk it should be good to remove master it will let : |
@ibuziuk won't it break Che 7 workspaces on prod-preview, will it? I assume prod-preview is updated on each commit |
@garagatyi sure what I mean is that this PR should not be merged until relevant changes are done in the rh-che |
Please note that I added CircleCI and Eclipse sign off check to required checks for PRs. |
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.
actually, I checked che.openshift.io stacks and we use the master version of theia in none. @mmorhun +1 to merge and rollout to prod
@mmorhun I believe https://github.com/eclipse/che-plugin-registry/blob/master/v2/plugins/eclipse/che-theia/master/meta.yaml should be also removed now |
Signed-off-by: Mykola Morhun <mmorhun@redhat.com>
9140370
to
2258ddf
Compare
@ibuziuk thanks, I've updated PR. But that's kinda surprise, because when I created it, |
What does this PR do?
This PR deletes deprecated Theia editor of
master
version.This will help to avoid confusion between Theia editor versions.
Instead of deleted
master
version, Theia with versionnext
should be used.