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

[Project Svelte] Deprecation Introduction Hitlist #15062

Open
chadhietala opened this Issue Mar 24, 2017 · 9 comments

Comments

Projects
None yet
5 participants
@chadhietala
Member

chadhietala commented Mar 24, 2017

Wanted

We need to know in which versions of Ember these deprecations were introduced. Please simply report the version a specific id was introduced. To ensure that we don't have overlapping efforts, please state what deprecation you are investigating before you do the work.

Why

With the eventual introduction of Svelte builds, there will be a mechanism to say "strip out any code for deprecated features from 2.0 to 2.8". However, to enable this range-based decision, we need both the start of the range.

How

Ways to find deprecations:

  • deprecate() method in the codebase
  • [DEPRECATE] and [DEPRECATION] tags in the CHANGELOG.

Deprecation IDs

  • container-lookup-factory
  • ember-application.injected-container
  • ember-application.app-instance-registry
  • ember-application.registry-resolver-as-function
  • ember-application.app-instance-container
  • ember-application.app-initializer-initialize-arguments
  • ember-application.validate-type
  • ember-debug.deprecate-options-missing
  • ember-debug.deprecate-id-missing
  • ember-debug.deprecate-until-missing
  • ember-debug.warn-options-missing
  • ember-debug.warn-id-missing
  • ember-views.component.defaultLayout
  • ember-routing-views.controller-wrapped-param
  • ember-htmlbars.make-bound-helper
  • ember-htmlbars.ember-handlebars-safestring
  • ember-metal.binding
  • ember-views.did-init-attrs
  • ember-metal.required
  • ember-metal.observer-argument-order
  • ember-metal.immediate-observer
  • ember-views.render-double-modify
  • ember-routing.router-resource
  • ember-routing.top-level-render-helper
  • ember-runtime.controller-proxy
  • ember-runtime.action-handler-_actions
  • ember-runtime.enumerable-contains
  • ember-runtime.will-merge-mixin
  • ember-runtime.frozen-copy
  • ember-runtime.enumerable-contains
  • ember-runtime.freezable-init
  • ember-string-utils.fmt
  • ember-template-compiler.deprecate-render-model
  • ember-template-compiler.deprecate-render
  • ember-template-compiler.transform-input-on-to-onEvent.dynamic-value
  • ember-template-compiler.transform-input-on-to-onEvent.no-action
  • ember-template-compiler.transform-input-on-to-onEvent.normalized-on
  • ember-template-compiler.transform-old-binding-syntax
  • ember-testing.test-waiters
  • ember-views.lifecycle-hook-arguments
  • ember-views.render-to-element
  • ember-views.did-init-attrs
  • ember-metal.ember-k
  • ember-metal.ember-backburner
@Serabe

This comment has been minimized.

Show comment
Hide comment
@Serabe

Serabe Mar 24, 2017

Member

Is the property still since?

Member

Serabe commented Mar 24, 2017

Is the property still since?

@Serabe

This comment has been minimized.

Show comment
Hide comment
@Serabe

Serabe Mar 24, 2017

Member

I'm working on this, I've gone through most of the changelog and still I have a few to solve.

I'll be uploading a PR soon with the part I get done by then.

Member

Serabe commented Mar 24, 2017

I'm working on this, I've gone through most of the changelog and still I have a few to solve.

I'll be uploading a PR soon with the part I get done by then.

@chadhietala

This comment has been minimized.

Show comment
Hide comment
@chadhietala

chadhietala Mar 24, 2017

Member

@Serabe We basically just need the mapping from deprecation ID to version. We are going to have deprecation flags that take a semver version. This semver version will get turned into a bool based on what the consuming app says it supports.

Member

chadhietala commented Mar 24, 2017

@Serabe We basically just need the mapping from deprecation ID to version. We are going to have deprecation flags that take a semver version. This semver version will get turned into a bool based on what the consuming app says it supports.

@Serabe

This comment has been minimized.

Show comment
Hide comment
@Serabe

Serabe Mar 24, 2017

Member

If you just need the mapping, this is the list:

  • container-lookup-factory not found in code
  • ember-application.app-initializer-initialize-arguments 2.1.0
  • ember-application.app-instance-container 2.1
  • ember-application.app-instance-registry 2.1
  • ember-application.injected-container 2.3.0
  • ember-application.registry-resolver-as-function 2.3.0
  • ember-application.validate-type 2.0.0
  • ember-debug.deprecate-id-missing 2.1
  • ember-debug.deprecate-options-missing 2.1
  • ember-debug.deprecate-until-missing 2.1
  • ember-debug.warn-id-missing 2.1
  • ember-debug.warn-options-missing 2.1
  • ember-htmlbars.ember-handlebars-safestring 2.8.0 (included before but reverted).
  • ember-htmlbars.make-bound-helper 2.0
  • ember-metal.binding 2.7
  • ember-metal.ember-backburner 2.7.0
  • ember-metal.ember-k 2.12.0
  • ember-metal.immediate-observer 2.0
  • ember-metal.observer-argument-order 2.0
  • ember-metal.required 2.0
  • ember-routing-views.controller-wrapped-param 2.0
  • ember-routing.router-resource 2.0
  • ember-routing.top-level-render-helper 2.11
  • ember-runtime.action-handler-_actions 2.0
  • ember-runtime.controller-proxy 2.0
  • ember-runtime.enumerable-contains 2.7
  • ember-runtime.freezable-init 2.0
  • ember-runtime.frozen-copy 2.0
  • ember-runtime.will-merge-mixin 2.0.
  • ember-string-utils.fmt 2.0
  • ember-template-compiler.deprecate-render 2.11
  • ember-template-compiler.deprecate-render-model 2.6
  • ember-template-compiler.transform-input-on-to-onEvent.dynamic-value 2.1
  • ember-template-compiler.transform-input-on-to-onEvent.no-action 2.1
  • ember-template-compiler.transform-input-on-to-onEvent.normalized-on 2.1
  • ember-template-compiler.transform-old-binding-syntax 2.1
  • ember-testing.test-waiters 2.7
  • ember-views.component.defaultLayout 2.1
  • ember-views.did-init-attrs 2.6
  • ember-views.lifecycle-hook-arguments 2.12
  • ember-views.render-double-modify 2.0
  • ember-views.render-to-element 2.11

If you don't need the actual since option, shall I close #15064 ?

Member

Serabe commented Mar 24, 2017

If you just need the mapping, this is the list:

  • container-lookup-factory not found in code
  • ember-application.app-initializer-initialize-arguments 2.1.0
  • ember-application.app-instance-container 2.1
  • ember-application.app-instance-registry 2.1
  • ember-application.injected-container 2.3.0
  • ember-application.registry-resolver-as-function 2.3.0
  • ember-application.validate-type 2.0.0
  • ember-debug.deprecate-id-missing 2.1
  • ember-debug.deprecate-options-missing 2.1
  • ember-debug.deprecate-until-missing 2.1
  • ember-debug.warn-id-missing 2.1
  • ember-debug.warn-options-missing 2.1
  • ember-htmlbars.ember-handlebars-safestring 2.8.0 (included before but reverted).
  • ember-htmlbars.make-bound-helper 2.0
  • ember-metal.binding 2.7
  • ember-metal.ember-backburner 2.7.0
  • ember-metal.ember-k 2.12.0
  • ember-metal.immediate-observer 2.0
  • ember-metal.observer-argument-order 2.0
  • ember-metal.required 2.0
  • ember-routing-views.controller-wrapped-param 2.0
  • ember-routing.router-resource 2.0
  • ember-routing.top-level-render-helper 2.11
  • ember-runtime.action-handler-_actions 2.0
  • ember-runtime.controller-proxy 2.0
  • ember-runtime.enumerable-contains 2.7
  • ember-runtime.freezable-init 2.0
  • ember-runtime.frozen-copy 2.0
  • ember-runtime.will-merge-mixin 2.0.
  • ember-string-utils.fmt 2.0
  • ember-template-compiler.deprecate-render 2.11
  • ember-template-compiler.deprecate-render-model 2.6
  • ember-template-compiler.transform-input-on-to-onEvent.dynamic-value 2.1
  • ember-template-compiler.transform-input-on-to-onEvent.no-action 2.1
  • ember-template-compiler.transform-input-on-to-onEvent.normalized-on 2.1
  • ember-template-compiler.transform-old-binding-syntax 2.1
  • ember-testing.test-waiters 2.7
  • ember-views.component.defaultLayout 2.1
  • ember-views.did-init-attrs 2.6
  • ember-views.lifecycle-hook-arguments 2.12
  • ember-views.render-double-modify 2.0
  • ember-views.render-to-element 2.11

If you don't need the actual since option, shall I close #15064 ?

@andyhot

This comment has been minimized.

Show comment
Hide comment
@andyhot

andyhot Mar 24, 2017

From the work (https://gist.github.com/andyhot/0c574101afc3ff25f95469453447930f) i had done for #14811, here's my (partial) findings:

  • ember-application.registry-resolver-as-function 2.3.0
  • container-lookupFactory 2.12.0
  • ember-application.injected-container 2.3.0
  • ember-metal.ember-k 2.12.0
  • ember-metal.ember-backburner 2.7.0
  • ember-metal.ember-create 1.13.3
  • ember-metal.ember.keys 1.13.3
  • ember-application.app-instance-container 2.1.0
  • ember-application.app-initializer-initialize-arguments 2.1.0
  • ember-application.validate-type 1.13.0
  • ember-views.component.defaultLayout 2.7.0
  • ember-routing-views.controller-wrapped-param 2.2.0
  • ember-htmlbars.make-bound-helper 1.13.6
  • ember-htmlbars.ember-handlebars-safestring 2.8.0
  • ember-routing.router-resource 2.0.0
  • ember-routing.top-level-render-helper 2.11.0

The differences I see with @Serabe 's findings:

  • I had tracked ember-metal.ember-create and ember-metal.ember.keys though those do not appear in this issue. This might be because they were declared with deprecateFunc, so may have been missed by @chadhietala
  • I had tracked container-lookupFactory and also included ember-application.app-instance-container, ember-routing.router-resource, ember-routing.top-level-render-helper in more details (though it's basically the same versions, i.e. 2.11.0 instead of 2.11)
  • I have different versions for ember-application.validate-type, ember-views.component.defaultLayout, ember-routing-views.controller-wrapped-param and ember-htmlbars.make-bound-helper so someone should take another look at those

fyi, all of the versions i had found were from tracking down the tagged commits hence the more detail (though i can just be plain wrong in same cases where code was moved to different files)

andyhot commented Mar 24, 2017

From the work (https://gist.github.com/andyhot/0c574101afc3ff25f95469453447930f) i had done for #14811, here's my (partial) findings:

  • ember-application.registry-resolver-as-function 2.3.0
  • container-lookupFactory 2.12.0
  • ember-application.injected-container 2.3.0
  • ember-metal.ember-k 2.12.0
  • ember-metal.ember-backburner 2.7.0
  • ember-metal.ember-create 1.13.3
  • ember-metal.ember.keys 1.13.3
  • ember-application.app-instance-container 2.1.0
  • ember-application.app-initializer-initialize-arguments 2.1.0
  • ember-application.validate-type 1.13.0
  • ember-views.component.defaultLayout 2.7.0
  • ember-routing-views.controller-wrapped-param 2.2.0
  • ember-htmlbars.make-bound-helper 1.13.6
  • ember-htmlbars.ember-handlebars-safestring 2.8.0
  • ember-routing.router-resource 2.0.0
  • ember-routing.top-level-render-helper 2.11.0

The differences I see with @Serabe 's findings:

  • I had tracked ember-metal.ember-create and ember-metal.ember.keys though those do not appear in this issue. This might be because they were declared with deprecateFunc, so may have been missed by @chadhietala
  • I had tracked container-lookupFactory and also included ember-application.app-instance-container, ember-routing.router-resource, ember-routing.top-level-render-helper in more details (though it's basically the same versions, i.e. 2.11.0 instead of 2.11)
  • I have different versions for ember-application.validate-type, ember-views.component.defaultLayout, ember-routing-views.controller-wrapped-param and ember-htmlbars.make-bound-helper so someone should take another look at those

fyi, all of the versions i had found were from tracking down the tagged commits hence the more detail (though i can just be plain wrong in same cases where code was moved to different files)

@chadhietala

This comment has been minimized.

Show comment
Hide comment
@chadhietala

chadhietala Mar 24, 2017

Member

@andyhot and @Serabe awesome! @Serabe I think we should keep your PR open still, I'm not sure if @rwjblue wants to do the since field in the deprecation. I don't know if it's needed with the thing I wrote.

Member

chadhietala commented Mar 24, 2017

@andyhot and @Serabe awesome! @Serabe I think we should keep your PR open still, I'm not sure if @rwjblue wants to do the since field in the deprecation. I don't know if it's needed with the thing I wrote.

@chadhietala

This comment has been minimized.

Show comment
Hide comment
@chadhietala

chadhietala Mar 24, 2017

Member

So I think I need verification on these @rwjblue and if we should just go cleanup all the 1.13.0 flags. There is also some untils that have expired.

Member

chadhietala commented Mar 24, 2017

So I think I need verification on these @rwjblue and if we should just go cleanup all the 1.13.0 flags. There is also some untils that have expired.

@rwjblue

This comment has been minimized.

Show comment
Hide comment
@rwjblue

rwjblue Mar 24, 2017

Member

yep, all of those can can be cleaned up

Member

rwjblue commented Mar 24, 2017

yep, all of those can can be cleaned up

@Serabe

This comment has been minimized.

Show comment
Hide comment
@Serabe

Serabe Jul 13, 2017

Member

Can this be closed?

Member

Serabe commented Jul 13, 2017

Can this be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment