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

Added Baboon Facts #326

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

Added Baboon Facts #326

wants to merge 4 commits into from

Conversation

anton-tr3
Copy link

No description provided.

@mpentler
Copy link
Collaborator

Hi @AntonTru3 ! Any chance you can take the latest codebase and try submitting these again? May also want to check your indents etc for consistency. Thanks!

@mpentler
Copy link
Collaborator

mpentler commented Dec 2, 2018

@antontru3 are you still around? Shall we get this fixed up? Happy to pull it when it’s all sorted.

animalfacts.py Outdated
@@ -227,6 +227,7 @@ def check_comment_for_animal(comment, reddit):
botengine('atlantic puffin', '\spuffins?\s', reddit, ATLANTIC_PUFFIN_FACTS, comment)
botengine('honeybadger', '\shoney badgers?\s', reddit, HONEYBADGER_FACTS, comment) # Needs to be here out of order so that it gets picked up before regular badger
botengine('badger', '\sbadgers?\s', reddit, BADGER_FACTS, comment)
botengine('baboon', '\sbaboon?\s', reddit, BABOON_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.

Mind making this 4 spaces instead of a tab, please? Sorry!

@mpentler
Copy link
Collaborator

mpentler commented Jan 2, 2019

Tabs everywhere instead of spaces. All we need is a rebase and a quick check of your additions and it's good to go.

Corrected some spellings, grammar, and indentation issues. Also a comma.
Corrected animal order (Baboon)
@mpentler
Copy link
Collaborator

I’ve fixed all of the issues with the additions, but there’s the conflicts to sort too. I’ll save that for another day.

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