-
-
Notifications
You must be signed in to change notification settings - Fork 185
General example changes #2581
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
General example changes #2581
Conversation
This should fix #2518 |
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.
LGTM 👍
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.
This PR needs a lint run to make CI happy
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 had already tested all these examples in the last reviewing round that I did. The code changes LGTM.
I will go ahead and merge this in now, though there is always scope for further improvements in future (in hopefully smaller and more focussed PRs).
Thanks for contributing! 🎉
* scaletest.py fixes * example fixes * format fix * format fixes * liquid example fix * format fic * removed time import * fix 1 * update example * droid problem fixes * alien example update * chimp example * removed durk from buildconfig * Revert "removed durk from buildconfig" This reverts commit d5897be. * Revert "chimp example" This reverts commit 5ed783e. * chimp tutorial update * format fix * playmus fix * Further aliens.py cleanups * Further scaletest.py cleanups * Update examples/aliens.py * formatting * scaletest fix using irl key access * clock scaletest * revert changes resizing_new --------- Co-authored-by: Dan Lawrence <danintheshed@gmail.com>
Hi, I took a look at all examples, and made some fixes.
aacircle.py
:aliens.py
:all
toall_sprite
, becauseall
is a native python function.depth
argumentpygame.SCALED
because on Windows, there is a problem with FULLSCREEN modechimp.py
:RenderPlain
because it's deprecateddropevent.py
:playmus.py
:pygame.freetype
topygame.Font
resizing_new.py:renamed a variable because it's not intuitiveRemoval plan is being discussed
scaletest.py
:liquid.py
:time.sleep
so it usespygame.Clock