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

feat(cloud.project): remove io stream routes #307

Merged
merged 1 commit into from
Oct 16, 2020

Conversation

marie-j
Copy link
Contributor

@marie-j marie-j commented Sep 3, 2020

BREAKING CHANGE: OvhApiCloudProjectIo and sub services are removed

Signed-off-by: Marie JONES <[email protected]>
@antleblanc antleblanc added the breaking-change Introduce a BREAKING CHANGE label Sep 14, 2020
@antleblanc
Copy link
Contributor

ovh/manager#3573 has been merged and live in production.

@marie-j This BREAKING CHANGE is not mandatory to patch the production environment right?
(except that it will reduce the bundle size).

Maybe we can consider creating a new branch where we can group a set of breaking changes
like removing deprecated services at once (like #306) and introduce tree shaking for instance.

cc @cbourgois

@marie-j
Copy link
Contributor Author

marie-j commented Sep 14, 2020

ovh/manager#3573 has been merged and live in production.

@marie-j This BREAKING CHANGE is not mandatory to patch the production environment right?
(except that it will reduce the bundle size).

Maybe we can consider creating a new branch where we can group a set of breaking changes
like removing deprecated services at once (like #306) and introduce tree shaking for instance.

cc @cbourgois

@antleblanc yes it is not mandatory to patch environment execution, this is to clean up this lib and avoid declaration no longer needed

@antleblanc antleblanc self-assigned this Oct 16, 2020
@antleblanc antleblanc merged commit e03b2dc into master Oct 16, 2020
@antleblanc antleblanc deleted the feat/remove-io-stream branch October 16, 2020 07:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking-change Introduce a BREAKING CHANGE
Development

Successfully merging this pull request may close these issues.

4 participants