Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Rename factory plugin to workspace plugin #592

Merged
merged 1 commit into from
Jan 9, 2020

Conversation

vitaliy-guliy
Copy link
Contributor

What does this PR do?

Renames the factory plugin to workspace plugin.

What issues does this PR fix or reference?

eclipse-che/che#14653

Copy link
Contributor

@mmorhun mmorhun left a comment

Choose a reason for hiding this comment

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

LGTM

@dmytro-ndp
Copy link
Contributor

crw-ci-test

@gorkem
Copy link

gorkem commented Jan 6, 2020

Can this change break any devfiles?

@benoitf
Copy link
Contributor

benoitf commented Jan 6, 2020

@gorkem I would say no as it's more 'internal renaming', it's not exposed in devfiles

@vitaliy-guliy
Copy link
Contributor Author

crw-ci-test

@benoitf
Copy link
Contributor

benoitf commented Jan 8, 2020

@vitaliy-guliy can you rebase code (to fix ci-build check ?)

@vitaliy-guliy vitaliy-guliy force-pushed the rename-factory-plugin branch from ae680a6 to 67725ae Compare January 8, 2020 10:10
@vitaliy-guliy
Copy link
Contributor Author

@benoitf done

@benoitf
Copy link
Contributor

benoitf commented Jan 8, 2020

thx

@vitaliy-guliy
Copy link
Contributor Author

crw-ci-test

2 similar comments
@vitaliy-guliy
Copy link
Contributor Author

crw-ci-test

@vitaliy-guliy
Copy link
Contributor Author

crw-ci-test

@RomanNikitenko
Copy link
Member

@vitaliy-guliy
FYI eclipse-che/che#15613

@artaleks9
Copy link

artaleks9 commented Jan 9, 2020

The test failure is now expected. @rhopp is working on a solution right now (issue: eclipse-che/che#15586).
PR can be merged.

@vitaliy-guliy
Copy link
Contributor Author

@RomanNikitenko @artaleks9 thanks!

@vitaliy-guliy vitaliy-guliy merged commit e148767 into master Jan 9, 2020
@vitaliy-guliy vitaliy-guliy deleted the rename-factory-plugin branch January 9, 2020 09:59
vinokurig pushed a commit that referenced this pull request Apr 6, 2021
Signed-off-by: Mykhailo Kuznietsov <[email protected]>

Co-authored-by: Mykhailo Kuznietsov <[email protected]>
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.

10 participants