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.

Sign up for GitHub

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

Jump to bottom

Readme example seems a bit wonky #2

Closed
padowi opened this issue Sep 27, 2013 · 3 comments
Closed

Readme example seems a bit wonky #2

padowi opened this issue Sep 27, 2013 · 3 comments
Assignees
@sujaymansingh

Comments

@padowi
Copy link

padowi commented Sep 27, 2013

The decorator example using

@dudebot.message_must_begin_with_attr("blah")

doesn't really make sense to me. How is "blah" linked to secret_key ("something")?
@sujaymansingh
Copy link
Owner

Ah, yes, that doesn't make much sense.
I suspect I left blah in as a placeholder while I thought of an example :)
I'll change it for a more meaningful example.

sujaymansingh added a commit that referenced this issue Sep 28, 2013
@sujaymansingh
[#2] Change example used for message_must_begin_with_attr
1bd0c3a 
The initial example made no sense (and was incorrect).
@sujaymansingh
Copy link
Owner

Does this one make more sense?

@ghost ghost assigned sujaymansingh Sep 28, 2013
@sujaymansingh sujaymansingh reopened this Sep 28, 2013
@padowi
Copy link
Author

padowi commented Sep 28, 2013

Yes, I think this is much clearer now :)

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

@sujaymansingh sujaymansingh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@padowi @sujaymansingh