Skip to content

Downstream testing CI changes#154

Merged
Samuel-amap merged 7 commits into
mainfrom
ensure_downstream_main_is_used_for_CI
Sep 19, 2025
Merged

Downstream testing CI changes#154
Samuel-amap merged 7 commits into
mainfrom
ensure_downstream_main_is_used_for_CI

Conversation

@Samuel-amap

Copy link
Copy Markdown
Collaborator
  • Dev branch removal
  • Fail action unless the tests check against dependencies main/master (enables testing with a downstream package PR, and ensures dependencies are in a 'releasable' state before tests pass).

@Samuel-amap

Copy link
Copy Markdown
Collaborator Author

Behaviour works as intended.

A couple of points that might need some consideration :

  • We might want to point at the latest release, occasionally, instead of testing against the main/master branches of dependencies.
  • If a version incompatibility is detected, the integration check is successful. It is understandable behaviour, but is this always desirable ? I think this doesn't cause a problem because it'll be caught when working on and testing the downstream dependency directly in this kind of scenario. There might be other side-effects I'm not taking into consideration that cause workflow issues, though.

@Samuel-amap Samuel-amap merged commit af10d28 into main Sep 19, 2025
14 of 22 checks passed
@Samuel-amap Samuel-amap deleted the ensure_downstream_main_is_used_for_CI branch September 19, 2025 15:33
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.

Improve the dependency testing system: avoid relying on a dev branch in the repositories. Non-steady-state models

1 participant