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

Update PyInstaller to 6.3 to fix missing imports in packaging #557

Merged
merged 12 commits into from
Jan 9, 2024

Conversation

rly
Copy link
Collaborator

@rly rly commented Jan 8, 2024

Fix #559, closes #556, closes #558, closes #560

@rly rly changed the title Update nwb-guide.spec Attempty pyflask tests fix Jan 8, 2024
@rly rly marked this pull request as draft January 8, 2024 19:03
@rly rly changed the title Attempty pyflask tests fix Fix issue with missing imports in packaging during pyflask tests Jan 8, 2024
Test using latest PyInstaller
@CodyCBakerPhD
Copy link
Collaborator

@rly CI is passing now! Let me know when you're done cleaning up anything and ready for review

Thanks for tackling this so quickly

@CodyCBakerPhD CodyCBakerPhD changed the base branch from fix_pyflask_tests to main January 9, 2024 01:58
@rly rly changed the title Fix issue with missing imports in packaging during pyflask tests Update PyInstaller to 6.3 to fix missing imports in packaging Jan 9, 2024
@rly
Copy link
Collaborator Author

rly commented Jan 9, 2024

This PR now updates PyInstaller from 5.13.0 to the latest version, 6.3.0. This version includes a few breaking changes: https://pyinstaller.org/en/stable/CHANGES.html#id9 . I don't see anything obvious that would affect GUIDE. @CodyCBakerPhD @garrettmflynn please review these changes to PyInstaller to make sure nothing might break because of them.

@rly rly marked this pull request as ready for review January 9, 2024 07:32
@rly
Copy link
Collaborator Author

rly commented Jan 9, 2024

I also removed testing of the PyFlask build and distributable on ubuntu and removed that as a required test for merging PRs.

@rly rly enabled auto-merge (squash) January 9, 2024 07:47
@rly rly disabled auto-merge January 9, 2024 07:47
@CodyCBakerPhD CodyCBakerPhD enabled auto-merge (squash) January 9, 2024 15:50
@CodyCBakerPhD
Copy link
Collaborator

Thanks a bunch @rly !

@CodyCBakerPhD CodyCBakerPhD merged commit d932295 into main Jan 9, 2024
10 checks passed
@CodyCBakerPhD CodyCBakerPhD deleted the fix_pyflask_tests2 branch January 9, 2024 16:21
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.

PyFlask failures - January 2024
2 participants