Conversation
afc4cbe
to
741fc96
Compare
Hey @dwickern I'm willing to help here ! Do you need me to ping you on slack ? |
@bartocc I'd appreciate the help! I made this branch to get things started for anyone who wants to contribute. We're in slack if you have any questions |
…corators # Conflicts: # types/ember-decorators/index.d.ts
} | ||
|
||
declare module 'ember-decorators/object' { | ||
export function computed(...keys: string[]): MethodDecorator; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs to be a PropertyDecorator
as well
export function computed(...dependentKeys: string[]): PropertyDecorator;
export function computed(target: any, propertyKey: string | symbol): void;
@dwickern |
I'm not sure this issue is relevant any more given that ember decorators are written in TS now. @pzuraq ? |
ember-decorators are not actually written in TS, they are compatible now. If we can add the typings there I’m completely ok with it, either way |
Yeah, we should close this issue. You can check the pull requests on the specific repositories I opened over the weekend! |
I'm creating this as a starting point for
ember-decorators
typings. I'm putting the typings here to leverage DT's test framework. Ultimately these typings belong in the ember-decorators repo.I've found that typescript decorators are severely limited in what they can express. I'll try to document some of those issues and possible workarounds.
I based this branch on the
new-object-model
branch since the typings didn't work on master (withoutnoImplicitAny
or some extra type annotations).