-
Notifications
You must be signed in to change notification settings - Fork 344
N16: ggez #385
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
N16: ggez #385
Conversation
It's probably too long, and it needs a cool picture. Hmmmmm.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no clue how this branch has gotten itself so confused - should be able to just cherry pick it to merge though, so not the end of the world 😄
Text looks good to me, just one minor formatting thing that's tripping up the linter:
Co-authored-by: Joe Clay <27cupsofcoffee@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 (made a few small style/fmt tweaks)
Thanks, merged! As for the other commits - force pushing the |
What the heck, I have no idea why this is dragging a bunch of other commits with it. I nuked and re-made my fork and branch from this repo and it's still doing it.
It's been one of those days.
Part of #365