Skip to content

Add CI Infrastructure - #2

Merged
julianadrianheine merged 2 commits into
releases/tri/0.10.5from
julianadrianheine/enable_ci
Jul 3, 2026
Merged

Add CI Infrastructure#2
julianadrianheine merged 2 commits into
releases/tri/0.10.5from
julianadrianheine/enable_ci

Conversation

@julianadrianheine

@julianadrianheine julianadrianheine commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds GitHub Actions CI and the local tooling needed to build and test the package
without a system-wide cyclonedds C installation.

Changes

.github/workflows/ci.yml
Runs the test suite on Python 3.11 and 3.12 on every push to main, releases/**,
and PRs targeting either. The wheel is cached by a hash of the source tree so the
cibuildwheel/Docker step is only re-run when the package content changes.

wheelhouse/build_cyclonedds_wheel.sh
Adapted from an external-clone approach to operate directly on the current repo.
Cloning and the CYCLONEDDS_COMMIT pin are removed; SOURCE_DATE_EPOCH is derived from
the current HEAD for reproducible output. The venv and cibuildwheel output are written
to a temp directory so the repo tree stays clean.

setup/install_prereq.sh
Local equivalent of the CI setup: installs Docker if missing, builds the wheel via
build_cyclonedds_wheel.sh, and creates a .venv with the wheel and test dependencies
installed. After running: .venv/bin/python local-ci.py --no-linter.

@julianadrianheine julianadrianheine self-assigned this Jun 30, 2026
@julianadrianheine

Copy link
Copy Markdown
Collaborator Author

I'm not entirely sure about this, but it makes sense to run the tests for both Python 3.11 and Python 3.12, since we are maintaining both versions, right? @frneer

@julianadrianheine
julianadrianheine requested a review from frneer June 30, 2026 17:50
@julianadrianheine
julianadrianheine force-pushed the julianadrianheine/enable_ci branch 2 times, most recently from 331f01e to 1c78a15 Compare July 1, 2026 12:54

@frneer frneer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job! I left some comments.

Comment thread requirements-dev.txt
Comment thread setup/install_prereq.sh Outdated
Comment thread setup/install_prereq.sh Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread setup/install_prereq.sh Outdated
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated
@julianadrianheine
julianadrianheine force-pushed the julianadrianheine/enable_ci branch from 231a77e to c190003 Compare July 1, 2026 17:56
@julianadrianheine
julianadrianheine requested a review from frneer July 1, 2026 17:56
Comment thread wheelhouse/build_cyclonedds_wheel.sh Outdated
@julianadrianheine
julianadrianheine force-pushed the julianadrianheine/enable_ci branch from c190003 to feac17d Compare July 2, 2026 15:10
Comment thread wheelhouse/build_cyclonedds_wheel.sh Outdated
@frneer

frneer commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

:LGTM: except one final comment.

@julianadrianheine
julianadrianheine merged commit 241b539 into releases/tri/0.10.5 Jul 3, 2026
2 checks passed
@julianadrianheine
julianadrianheine deleted the julianadrianheine/enable_ci branch July 3, 2026 11:21
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.

2 participants