Skip to content

Enable CI workflow, add healpy test coverage, clean up dev dependencies#12

Merged
VoxleOne merged 3 commits intomainfrom
copilot/implement-suggested-changes-status
Mar 24, 2026
Merged

Enable CI workflow, add healpy test coverage, clean up dev dependencies#12
VoxleOne merged 3 commits intomainfrom
copilot/implement-suggested-changes-status

Conversation

Copy link
Contributor

Copilot AI commented Mar 24, 2026

Implements the five suggested changes from STATUS.md to make SpinStep shipping-ready.

CI workflow (.github/workflows/ci.yml)

  • Moved from .github/workflows_disabled (inert) → .github/workflows/ci.yml (active)
  • Python matrix: 3.8['3.9', '3.10', '3.11', '3.12'] to match requires-python
  • Linter: flake8ruff check spinstep/ to match project tooling
  • Added permissions: contents: read (least-privilege)

Test coverage

  • Added TestGetUniqueRelativeSpins for the previously untested get_unique_relative_spins() HEALPix path
  • Uses pytest.importorskip("healpy") at method level to skip gracefully without affecting other tests in the module

Cleanup

  • Removed stale dev-requirements.txt entries (matplotlib, numba, cython, cupy, pycuda) — none used in core or tests
  • Fixed 3 pre-existing unused imports flagged by ruff (Optional/Union in discrete.py, node.py)

Test results: 51 collected, 46 passed, 5 skipped (4 CUDA + 1 healpy).


📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Copilot AI and others added 2 commits March 24, 2026 14:38
@VoxleOne VoxleOne marked this pull request as ready for review March 24, 2026 14:51
@VoxleOne VoxleOne merged commit 92a6f1f into main Mar 24, 2026
4 checks passed
@VoxleOne VoxleOne deleted the copilot/implement-suggested-changes-status branch March 24, 2026 15:59
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