Skip to content

Conversation

@jafranc
Copy link
Contributor

@jafranc jafranc commented Oct 16, 2025

In order to avoid long CI build when work-in-progress,

  • discard CI launch if semantics is ok but PR in draft mode
  • skip the GEOS-integration CI even if required as long as the flag test-geos-integration is not set
  • add a force-geos-ci flag to force it

to test new CI check:

  • draft clause should fail and not build nor CI-integration-GEOS (ci #636)
  • untouched protected part and not labeled should pass (ci #637)
  • modified protected and not labeld should fail (ci #640)
  • modified protected with label should pass (ci #641 *)
  • untouched protected part and force labeled should pass (ci #643)

*canceled by another push to but passed teh dispatch as expected

@jafranc jafranc self-assigned this Oct 16, 2025
@jafranc jafranc changed the title Testing CI adaptation ci: Testing CI adaptation Oct 16, 2025
@jafranc jafranc marked this pull request as ready for review October 16, 2025 11:26
@jafranc jafranc added the test-geos-integration Triggers the testing of geosPythonPackages import and integration in GEOS CI label Oct 16, 2025
@jafranc jafranc added force-geos-integration Force triggering of GEOS integration CI even if non req and removed test-geos-integration Triggers the testing of geosPythonPackages import and integration in GEOS CI labels Oct 16, 2025
@jafranc jafranc changed the title ci: Testing CI adaptation ci: CI shortened queue time Oct 16, 2025
@jafranc
Copy link
Contributor Author

jafranc commented Oct 16, 2025

@alexbenedicto I am unsure what maybe could be spared in the final stage, I let you guide me through

@jafranc jafranc changed the base branch from jafranc/ci/discard-draft to main October 16, 2025 18:04
@alexbenedicto
Copy link
Collaborator

I think it's great. Can we merge @jafranc ?

@jafranc
Copy link
Contributor Author

jafranc commented Oct 20, 2025

I think it's great. Can we merge @jafranc ?

Yes we can :)

@alexbenedicto alexbenedicto merged commit ffc89dc into main Oct 20, 2025
52 checks passed
@alexbenedicto alexbenedicto deleted the jafranc/ci/test-draft-clause branch October 20, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flag: ready for review force-geos-integration Force triggering of GEOS integration CI even if non req

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants