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

Python 3.7 support #60

Merged
merged 4 commits into from
Sep 18, 2019
Merged

Python 3.7 support #60

merged 4 commits into from
Sep 18, 2019

Conversation

HENDRIX-ZT2
Copy link
Contributor

@niftools/pyffi-reviewer

Overview

Replaces expressions that are deprecated in 3.7 by stuff that works in all versions. Also adds small enhancements / fixes.

Fixes Known Issues

none

Documentation

none

Testing

none

Manual

none

Automated

none

Additional Information

none

- replace `raise StopIteration` by `return`
https://www.python.org/dev/peps/pep-0479/#examples-of-breakage
- replace time.clock() by time.time() since we don't really care which time it is but the latter still works
We can keep the original formatting of attribute names. snakeCase broke access to enum attributes in the blender plugin.
add a setter function and implicit access to the x,y,z properties
@coveralls
Copy link

coveralls commented Sep 18, 2019

Coverage Status

Coverage decreased (-0.05%) to 57.826% when pulling 63aef5d on HENDRIX-ZT2:develop into 440c61c on niftools:develop.

@neomonkeus
Copy link
Member

Merging for now with known limitations around generators.

@neomonkeus neomonkeus merged commit 3bb308a into niftools:develop Sep 18, 2019
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.

None yet

3 participants