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

Imperative form? #6

Closed
arjan opened this issue Oct 24, 2016 · 3 comments
Closed

Imperative form? #6

arjan opened this issue Oct 24, 2016 · 3 comments

Comments

@arjan
Copy link
Owner

arjan commented Oct 24, 2016

Looking at the Erlang counterpart, I see they use the @decorate annotation, instead of @decorator that we use. @decorate is maybe a bit more clear, given that's an imperative, instead of a noun.

@decorator maybe implies that we are defining a decorator, while in fact we are saying that we want to apply the decorator to the function.

@vic what do you think?

@vic
Copy link
Contributor

vic commented Oct 24, 2016

@decorator maybe implies that we are defining a decorator, while in fact we are saying that we want to apply the decorator to the function.

Yep, I'd go with @decorate as you are applying a decorator to next function def.

@arjan
Copy link
Owner Author

arjan commented Oct 24, 2016

Done!

@arjan arjan closed this as completed Oct 24, 2016
@arjan
Copy link
Owner Author

arjan commented Oct 24, 2016

I think I should release 1.0 now, because I'm not going to change the API anymore probably.

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

No branches or pull requests

2 participants