Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

Adapt to new plugin notation #1390

Closed
wants to merge 1 commit into from
Closed

Adapt to new plugin notation #1390

wants to merge 1 commit into from

Conversation

garagatyi
Copy link

What does this PR do?

Do not merge until PR eclipse-che/che#13204 is in the version of upstream RhChe uses.
Note: this PR doesn't adapt resources prefetcher by @davidfestal

What issues does this PR fix or reference?

How have you tested this PR?

Signed-off-by: Oleksandr Garagatyi <ogaragat@redhat.com>
@garagatyi garagatyi requested a review from ibuziuk April 28, 2019 07:59
@centos-ci
Copy link
Collaborator

@garagatyi The che server [build 1408] failed.

Link URL
console https://ci.centos.org/view/Devtools/job/devtools-rh-che-rh-che-prcheck-dev.rdu2c.fabric8.io/1408/console
artifacts http://artifacts.ci.centos.org/devtools/rhche/devtools-rh-che-rh-che-prcheck-dev.rdu2c.fabric8.io/1408/

If the build or deployment fails, the artifacts will not be present. Don't panic, just grab a towel.

@l0rd l0rd mentioned this pull request May 2, 2019
@ibuziuk ibuziuk changed the title WIP Adapt to new plugin notation Adapt to new plugin notation May 13, 2019
@ibuziuk
Copy link
Member

ibuziuk commented May 13, 2019

@amisevsk @davidfestal could you please review?

@ibuziuk
Copy link
Member

ibuziuk commented May 13, 2019

@amisevsk it looks like you have already done all the required changes and this PR can be safely closed, right?

@ibuziuk
Copy link
Member

ibuziuk commented May 13, 2019

probably we can just import the doc changes in dev-guide.adoc

@amisevsk
Copy link
Collaborator

@ibuziuk Yeah I didn't notice this PR and reproduced a few of the changes. The dev-guide changes are still great to merge though.

@amisevsk
Copy link
Collaborator

Created #1407

@ibuziuk
Copy link
Member

ibuziuk commented May 14, 2019

Thank you @amisevsk
I believe now we can safely close this PR

@ibuziuk ibuziuk closed this May 14, 2019
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