Skip to content
This repository was archived by the owner on Mar 31, 2020. It is now read-only.

Blue Buckets #5

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

Conversation

intendednull
Copy link

Code jam 4 submission.

@intendednull
Copy link
Author

intendednull commented Mar 4, 2019

Unfortunately we were a man down, and unable to finish.

I ran into a bug last minute, and wasn't able to resolve it in time. Had to revert master to a older commit in order to be runnable (you can see that panic setting in with the commit message lengths).

I resolved the bug an hour later, and pushed the changes to staging. That is the code that actually starts to implement our idea.

Fair warning, there's no end. Enjoy the pictures though ;)

@lemonsaurus
Copy link
Member

At first glance, this is a very impressive pull request. The code demonstrates mastery of many of Pythons more advanced features, and even has stuff like unit tests. It's well documented, and of very high quality.

We will be testing the actual application for the first time live on the stream on Sunday, so this is just a superficial look.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants