Skip to content

Comments

test: Add import tests and workflow job for core dependencies only#75

Merged
XkunW merged 7 commits intodevelopfrom
kwaak/check-imports-core-only-0f963b0c
Mar 18, 2025
Merged

test: Add import tests and workflow job for core dependencies only#75
XkunW merged 7 commits intodevelopfrom
kwaak/check-imports-core-only-0f963b0c

Conversation

@amrit110
Copy link
Member

@amrit110 amrit110 commented Mar 18, 2025

Summary

  • Added a test in tests/test_imports.py to ensure that certain modules in the vec_inf package are importable with only the core dependencies installed.
  • Updated .github/workflows/unit_tests.yml to check that all modules are importable without development dependencies across various Python versions.

Changes

  • tests/test_imports.py: Contains tests for importability of vec_inf.cli modules.
  • .github/workflows/unit_tests.yml: Introduces a test for core dependency verification without dev tools.

Notes

  • Ensures CI checks module importability with core dependencies only, maintaining resilient package setups without being reliant on dev dependencies.

@amrit110 amrit110 changed the title test: ensure module imports with core package only test: add import tests to verify core package importability Mar 18, 2025
@amrit110 amrit110 marked this pull request as draft March 18, 2025 18:32
@amrit110 amrit110 changed the title test: add import tests to verify core package importability test: Add import tests and workflow job for core dependencies only Mar 18, 2025
@amrit110 amrit110 marked this pull request as ready for review March 18, 2025 19:12
@amrit110 amrit110 requested a review from XkunW March 18, 2025 19:12
@XkunW XkunW merged commit 3794604 into develop Mar 18, 2025
6 checks passed
@XkunW XkunW deleted the kwaak/check-imports-core-only-0f963b0c branch March 18, 2025 20:58
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