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

Deprecate Ember.deprecate test function argument #11933

Conversation

KevinTCoughlin
Copy link

This PR deprecates:

  • Ember.deprecate when test is supplied as a function

Partially resolving #11898. As mentioned, it should be kept around until 2.0.0.

Would like fully resolve #11898, however unsure how to handle the assert, etc. cases. Could use some guidance as this is my first contribution. Thanks!

/cc @mmun @rwjblue

@rwjblue
Copy link
Member

rwjblue commented Jul 31, 2015

This looks good, thank you for working on it!

Moving the deprecation to around here would make this PR handled both Ember.deprecate and Ember.warn (since both use the same logic). Alternatively, you could take the same tactic that you did in ember-debug/deprecate for ember-debug/warn.

As for Ember.assert, you can take a look here for where to deprecate.

@KevinTCoughlin
Copy link
Author

Moving the deprecation to around here would make this PR handled both Ember.deprecate and Ember.warn (since both use the same logic).

@rwjblue: I'd like to do that & may be missing an obvious workaround, but deprecate and warn will then be recursive unless I add a way to short-circuit invoke.

@KevinTCoughlin KevinTCoughlin deleted the deprecate-test-as-function branch August 23, 2015 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate and remove function argument in Ember.assert/deprecate/etc
2 participants