Skip to content

Update CI to follow how we run tests#228

Open
furtib wants to merge 9 commits into
Ericsson:mainfrom
furtib:ci-rework
Open

Update CI to follow how we run tests#228
furtib wants to merge 9 commits into
Ericsson:mainfrom
furtib:ci-rework

Conversation

@furtib

@furtib furtib commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Why:
The way CI works and how tests are run locally (#218) shifted dramatically, so I updated the CI to correctly reflect how these tests were supposed to be run locally.

What:

  • Merged lint/unit/FOSS CI into 1 (no reason to have them so fragmented)
  • used pytest for running Python tests
  • added bazel test //... to CI
  • Replaced environment setup with micromamba
  • Added a patch to the FOSS test yaml-cpp -- newer versions of gcc/clang failed to build the project

Addresses:
none

depends on: #233

@furtib furtib requested a review from Szelethus June 4, 2026 12:53
@furtib furtib self-assigned this Jun 4, 2026
@furtib furtib added the CI 📦 label Jun 4, 2026
@furtib furtib force-pushed the ci-rework branch 5 times, most recently from e7fe957 to ac54a71 Compare June 5, 2026 12:18
@Szelethus

Copy link
Copy Markdown
Contributor

Well, the new docs suggests source .ci/micromamba/init.sh, these jobs don't do that.

@furtib furtib force-pushed the ci-rework branch 9 times, most recently from 7349122 to 2a3ad45 Compare June 8, 2026 06:59
@furtib furtib marked this pull request as draft June 8, 2026 07:52
@furtib furtib force-pushed the ci-rework branch 7 times, most recently from ab23bc4 to bb3d53a Compare June 8, 2026 11:29
@furtib furtib marked this pull request as ready for review June 8, 2026 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants