-
Notifications
You must be signed in to change notification settings - Fork 97
Description
What is the requested feature?
Add a separate testing of geosPythonPackages import in GEOS to make sure each new version of geosPythonPackages can be used in GEOS.
This will also allow geosPythonPackages to have its own CI implementation to test every new PR wrt to GEOS integration before merging them.
Is your request related to a specific problem?
Currently, two linked issues exists in GEOS and geosPythonPackages CIs:
- GEOS CI does not have a clear method to identify issues that would occur when importing geosPythonPackages.
- geosPythonPackages cannot test if a PR that will be merged will have an impact in GEOS later.
Describe the solution you'd like
In GEOS:
CI now would include a Python integration test that:
- Runs alongside other GEOS CI jobs
- Tests that geosPythonPackages can be properly installed and used
- Validates key Python tools like
preprocess_xml
,format_xml
, andmesh-doctor
- Ensures Python packages can be imported correctly
In geosPythonPackages:
When creating a PR , a strict sequential process is enforced:
- Step 1: Standard CI Tests Run First
- Step 2: Label Check for
test-geos-integration
(Only After Standard Tests Pass) - Step 3: GEOS Integration Testing (Automatic Trigger)
- Step 4: Final Validation with all steps that pass (Standard CI + Label + GEOS tests)
Additional context
The geosPythonPackages repository will need a GEOS_INTEGRATION_TOKEN
secret with permissions to trigger workflows in the GEOS repository.