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

Add a few more jokes, change the jokes.json format a bit, and add a devpun.py script #3

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

Conversation

BMDan
Copy link

@BMDan BMDan commented May 1, 2019

More jokes

I'm hilarious. Even my mom thinks so.

jokes.json Format Changes

I noticed that most of jokes.json repeats its question and answer in the text, so I made the text an optional override for the question and answer. Support for this particular bit of prestidigitation in the JavaScript implementation is beyond my expertise.

I also sorted the keys and standardized their spacing via a quick round-trip through python -m json.tool.

devpun.py

Done in typed Python formatted with black and isort. Why? Because we might be writing jokes, but even we have standards, dammit!

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