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 Transform #1203

Merged
merged 1 commit into from
Sep 27, 2017
Merged

Remove Transform #1203

merged 1 commit into from
Sep 27, 2017

Conversation

aslakhellesoy
Copy link
Contributor

@aslakhellesoy aslakhellesoy commented Sep 21, 2017

As I've argued in #1190 we should remove Transform rather than deprecating it. My position is that deprecated functionality should satisfy all of the following:

  • work as before deprecation
  • provide users with an alternative API in the same version
  • no longer issue warnings when the alternative API is used

We can't do all of these in a 2.x release, so we should remove in 3.0, and not deprecate in 2.x.

@aslakhellesoy aslakhellesoy added this to the 3.0 milestone Sep 21, 2017
@ghost ghost assigned aslakhellesoy Sep 21, 2017
@mattwynne mattwynne merged commit 95f675d into master Sep 27, 2017
@lock
Copy link

lock bot commented Oct 24, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 24, 2018
@luke-hill luke-hill deleted the remove-transform branch February 14, 2019 15:38
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.

2 participants