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.

Sign up for GitHub

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

Jump to bottom

Remove "if 1:" pattern in tests #505

Merged

illume merged 5 commits into pygame:master from e1000:remove-if-1

Aug 24, 2018

e1000 commented Aug 24, 2018

remove many "if 1: in test
remove two unreachable code in the else-part of if 1: (not sure what the meaning of this other code was, so maybe we want to do something with it , but differently ?)
Some PEP8 wsp cleanup in display_test.py
The "if 1:" in run_tests__tests/ seem to make sense as intended dummy-code, so I left them

e1000 added 5 commits

August 23, 2018 16:10


          display_test.py : rm strange 'if 1:' indentation

d89c68f


          transform_test.py : rm strange 'if 1:' indentation

058100a


          test cleanup : rm dead code outside of 'if 1:'

4e28aeb


          Merge branch 'master' into remove-if-1

6f8eb16


          display_test.py : some PEP8 cleanup

37fe2fb

illume merged commit 3cee812 into pygame:master

illume mentioned this pull request

1.9.5 release notes. #561

Closed

4 tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment