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

Default 4 syllable keyword HelloJasper works much better for new users #493

Open
wants to merge 5 commits into
base: jasper-dev
Choose a base branch
from

Conversation

GeoBeBee
Copy link

This is my first ever contribution to open source, so please be kind? - I have lots of ideas... ;-)

What: The change is very simple: the default keyword in application.py is changed from "Jasper" to "HelloJasper" which I have found is far more reliable.

Why: I have found lots of people struggling with Jasper not recognising the keyword, and I guess some may just give up on it, which would be a pity after they got so far.

I have created the following and posted it in various places in the hope that people will understand the use of the keyword:
https://github.com/GeoBeBee/mySystem/blob/master/Jasper_keywords.txt

As a next change I might update jasper-client/client/populate.py to set the keyword..?

Keeping this forking, committing, pushing, branching simple to start with... ;-p
Thanks, Geoff

@GeoBeBee
Copy link
Author

The problem with the weather might be fixed by #483, another day...;-).
Keen to know if this Pull Request will be accepted once fixed...?

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling e579cc6 on GeoBeBee:jasper-dev into * on jasperproject:jasper-dev*.

@GeoBeBee
Copy link
Author

As far as I understand it I've done what I can.
coveralls seems to be automated, and I don't know what it means...

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.

2 participants