nvidia-holoscan / holoscan-cli / 39603405208

82%

Build Type

push

github

Committed by web-flow

Commit Message

Fix install --dev for SDK 5 Modules and clean up CI checks (#274)

* fix(install): link SDK 5 Module build trees with install --dev

install --dev looked only for the holoscan_<slug>_dev.py helper and .pth
pair that SDK 4 Module builds staged. The SDK 5 template never stages
them, so install --dev failed for every SDK 5 Module.

A Module created with --language cpp python writes its package and its
holoscan.modules metadata to <build>/python/lib, and the SDK resolves
holoscan.<slug> through that metadata. install --dev now finds these
build trees by their holoscan.modules entry points and installs a
holoscan-<slug>-dev.pth that puts python/lib on sys.path. SDK 4 hooks
and --uninstall work as before.

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* ci: drop the HoloHub wrapper integration job

HoloHub is deprecated. Its public main is now a module registry, and
the archive-4x branch that keeps the wrapper suite is frozen and pins
CLI 4.6.0. Testing the current CLI against that suite no longer covers
a supported workflow, so remove the job and its row in .github/CI.md.
The build job now waits only for pre-commit and the test matrix.

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* ci: keep Dependabot out of the Module template

The SDK 5 Module template's pyproject.toml holds Jinja conditionals, so
it is not valid TOML. Dependabot's pip scan stops at that file, which
also stops the dependency graph from refreshing. Exclude the packaged
templates from the pip ecosystem's manifest search.

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

Coverage Stats

28 of 30 new or added lines in 1 file covered. (93.33%)

2 existing lines in 1 file now uncovered.

4743 of 5761 relevant lines covered (82.33%)

0.82 hits per line