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

Remove deprecated factory methods #5203

Merged
merged 6 commits into from
May 29, 2017
Merged

Remove deprecated factory methods #5203

merged 6 commits into from
May 29, 2017

Conversation

mkuznyetsov
Copy link
Contributor

@mkuznyetsov mkuznyetsov commented May 25, 2017

What does this PR do?

Removed deprecated methods:

  • saveFactory(Iterator)
  • getImage()
  • getFactorySnippet()

What issues does this PR fix or reference?

#5060

Changelog

Removed deprecated methods from FactoryService

Release Notes

Docs PR

@mkuznyetsov mkuznyetsov added the kind/task Internal things, technical debt, and to-do tasks to be performed. label May 25, 2017
@mkuznyetsov mkuznyetsov self-assigned this May 25, 2017
@mkuznyetsov mkuznyetsov changed the title Remove deprecated saveFactory method from Swagger Remove deprecated saveFactory method May 25, 2017
@mkuznyetsov
Copy link
Contributor Author

Updated PR to remove deprecated method completely from the code

@codenvy-ci
Copy link

Build # 2674 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2674/ to view the results.

@codenvy-ci
Copy link

Build # 2685 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2685/ to view the results.

@mkuznyetsov mkuznyetsov changed the title Remove deprecated saveFactory method Remove deprecated factory method May 29, 2017
@skabashnyuk skabashnyuk added this to the 5.12.0 milestone May 29, 2017
@mkuznyetsov mkuznyetsov changed the title Remove deprecated factory method Remove deprecated factory methods May 29, 2017
@mkuznyetsov
Copy link
Contributor Author

Changed PR to remove all currently deprecated factory methods:
saveFactory(Iterator)
getImage()
getFactorySnippet()

@codenvy-ci
Copy link

Build # 2686 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2686/ to view the results.

@mkuznyetsov mkuznyetsov merged commit 55f6e17 into master May 29, 2017
@mkuznyetsov mkuznyetsov deleted the che-5060 branch May 29, 2017 14:28
@codenvy-ci
Copy link

Build # 2694 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2694/ to view the results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants