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

Prairie dog added #311

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Prairie dog added #311

wants to merge 2 commits into from

Conversation

rmartin5
Copy link

@rmartin5 rmartin5 commented Oct 2, 2018

No description provided.

animalfacts.py Outdated
@@ -2270,6 +2271,21 @@ def animalfactsbot(reddit):
'Platypuses is the correct plural form, although platypi and platypodes are also accepted.',
)

PRAIRIEDOG_FACTS = (
'Prairie dog\'s entire mating season is just an hour long.',
'They live in tight-knit groups called coteries.',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a reference to the animal

animalfacts.py Outdated
'Prairie dogs live in underground burrows which have a complex tunnel and warren system.',
'Prairie dogs have a range of different barks and chirps for communicating different messages.',
'The name Prairie dog was given due to their habitat (prairies/grasslands) and their warning calls which sound like barks.',
'They live 3 to 5 years in the wild and up to 10 years in captivity.',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a reference to the animal here.

@@ -329,6 +329,7 @@ def check_comment_for_animal(comment, reddit):
botengine('pig', '\spigs?\s', reddit, PIG_FACTS, comment)
botengine('pigeon', '\spigeons?\s', reddit, PIGEON_FACTS, comment)
botengine('platypus', '\splatypuse?s?\s', reddit, PLATYPUS_FACTS, comment)
botengine('prairie dog', '\sprairie dogs?s?\s', reddit, PRAIRIEDOG_FACTS, comment)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another one for #168 @joel-g so we can't really take this yet. Also do you want the tuple name as two words? I am so picky!

Fixed references
@rmartin5
Copy link
Author

Changed both reference issues

@mpentler
Copy link
Collaborator

Thanks, that’s a help. Just need to see if the other thing is a problem now.

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