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

Feature/my new joke #94

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

Conversation

dwshore
Copy link

@dwshore dwshore commented Jan 15, 2019

This addresses issue #1 . Adds some new dad jokes.

Copy link
Author

@dwshore dwshore left a comment

Choose a reason for hiding this comment

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

Good times!

@@ -33,6 +33,9 @@ struct JokeStore {
Joke(description: "Why don't programmers like being outside?", punchline: "Because there's too many bugs"),
Joke(description: "Why don't bachelors like Git?", punchline: "Because they are afraid to commit."),
Joke(description: "What does the father play with his son?", punchline: "Try Catch"),
Joke(description: "Be careful with functions that return a value...", punchline: "In fact, I would just a void them."),
Joke(description: "How did pirates collaborate before computers?", punchline: "Pier to pier networking"),
Joke(description: "Why does Yoda's code always crash?", punchline: "Because there is no try."),
Copy link
Author

Choose a reason for hiding this comment

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

For the Star Wars fans out there...

@dwshore
Copy link
Author

dwshore commented Jan 15, 2019

Have created a Travis CI account. Trying again.

Not sure why my commit created this, but hoping it's what is causing my PR's to fail the Travis CI checks.
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.

1 participant