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 and remove function argument in Ember.assert/deprecate/etc #11898

Closed
mmun opened this issue Jul 25, 2015 · 6 comments
Closed

Deprecate and remove function argument in Ember.assert/deprecate/etc #11898

mmun opened this issue Jul 25, 2015 · 6 comments

Comments

@mmun
Copy link
Member

mmun commented Jul 25, 2015

The following should throw a deprecation:

Ember.deprecate(msg, function() { return true; })

The message should be something like "Functional arguments are ambiguous with constructors. Please use !!Constructor for constructors, or an IIFE to compute the deprecation value. In a future version of Ember functions will be treated as truthy values instead of being executed."

Needs to be deprecated in stable (1.13), removed in beta (2.0).

@rodrigo-morais
Copy link

Hello @mmun and @rwjblue,

I'd love become me a new contributor, and I think this issue is a good one to start.
I can't find where I should work to solve this issue.

Would you mind giving me some tips?

Thanks.

@KevinTCoughlin
Copy link

I've attempted to partially resolve this issue in #11933. Would appreciate any advice for fully satisfying the issue and any problems with my PR. Thank you.

@e00dan
Copy link
Contributor

e00dan commented Sep 18, 2015

@rwjblue, @stefanpenner is it still possible to create pr with deprecation or you will just remove this functionality? @mmun has written that it should be remove in beta 2.0.

@mmun
Copy link
Member Author

mmun commented Sep 18, 2015

It's far too late for this, but we can do it now in preparation for Ember 3.0.

@kylemellander
Copy link

In researching this, it looks like @kuzirashi has done a lot of great work moving this along in [https://github.com//pull/12370](PR 12370). I would love to help move this forward if it needs more work, or have it closed if Kuzirashi's work finished it.

@rwjblue
Copy link
Member

rwjblue commented Nov 18, 2015

@kylemellander - Thanks for doing the research on this, I think this is closed!

@rwjblue rwjblue closed this as completed Nov 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants