Skip to content

Modernize CI, add dev dependencies - #94

Merged
adrien-laposta merged 3 commits into
simonsobs:mainfrom
sbiquard:sb/ci
Sep 28, 2026
Merged

adrien-laposta merged 3 commits into
simonsobs:mainfrom
sbiquard:sb/ci

Conversation

@sbiquard

@sbiquard sbiquard commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor
  • Drop unused CONDA_ENV/CACHE_NUMBER vars
  • Drop Fortran toolchain setup: camb ships prebuilt wheels, and pymaster does not need Fortran
  • Drop manual pip install numpy/scipy/healpy/...: these are already declared as soopercool dependencies in pyproject.toml
  • Drop unused coveralls install (never invoked)
  • Move flake8/pytest/pytest-cov into a dev dependency group in pyproject.toml instead of installing them ad hoc in the workflow
  • Bump actions/checkout to a recent version
  • Install everything via uv on Python 3.13
  • Delete .github/environment.yml: unreferenced by the workflow, likely dead
  • uv venv puts .venv inside the repo, so add it to flake8's exclude list in .flake8
  • Drop dead global declarations in mpi_utils.py: no assignments in that scope, so global does nothing there (newly flagged by pyflakes' F824 check added in pyflakes 3.3.0, which wasn't around under the old CI's pinned flake8<7.2 / pyflakes<3.3.0).

@sbiquard
sbiquard marked this pull request as ready for review September 17, 2026 15:36

@adrien-laposta adrien-laposta 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.

This sounds good, happy to merge

@adrien-laposta
adrien-laposta merged commit 2d44225 into simonsobs:main Sep 28, 2026
1 check passed
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