REL: 4.8.0#476
Merged
Merged
Conversation
Closed
Member
Thanks! I'll take some minutes to make that a triple-check. |
Member
|
A few warnings while creating the sdist: |
Member
|
Looking good to me on Python 3.7.4 / macOS 10.14.6 / packages from PyPI, modulo the NumPy deprecation warnings that you already noted. Full transcript follows. |
Member
|
Same tests with a Python 2.7.16 virtualenv. (Still looks good.) |
Member
|
One fairly harmless warning from |
Contributor
Author
|
Thanks @mdickinson ! I've uploaded the tarball to PyPI and I've tagged the release commit. |
Member
|
And as a final check, all the tests pass for me after installing from PyPI: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This time this has all the necessary changes ;-)
I've double-checked that tests pass in a new (Python 3.7) environment. There are some deprecation warnings (ticketed in #475) that should be looked at but aren't urgent. While testing, I also ran into enthought/enable#360, which was tricky to get round (my home machine runs Arch Linux, which doesn't ship with swig-3 any longer).
@mdickinson This is the follow-up from #472