-
Notifications
You must be signed in to change notification settings - Fork 89
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 Packaging Updates #3057
Python Packaging Updates #3057
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3057 +/- ##
========================================
Coverage 55.69% 55.69%
========================================
Files 1032 1032
Lines 87739 87739
========================================
Hits 48863 48863
Misses 38876 38876 ☔ View full report in Codecov by Sentry. |
@untereiner - It looks like we'll need to update some import names in the integrated tests. I'll give this a shot and see how if the CI integrated tests step passes. |
6ab546e
to
63273c9
Compare
63273c9
to
d1a3606
Compare
@cssherman there are integrated tested that are failing.. is it ok ? |
756832b
to
a671f76
Compare
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.
OK, this looks good to go now. Let's add it to the merge queue
a671f76
to
fe2bb4c
Compare
Note: Before merging this branch, we need to do 1) Merge the changes in the linked geosPythonPackages branch, 2) Remove the temporary branch name in the cmake, and 3) Re-run the tests |
fe2bb4c
to
5ff42aa
Compare
Updating geosPythonPackages --------- Co-authored-by: Christopher Sherman <[email protected]>
Associated geosPythonPackages PR: GEOS-DEV/geosPythonPackages#13