Skip to content

Modernise Packaging#110

Merged
matthewtownson merged 18 commits intoAOtools:mainfrom
matthewtownson:master
Apr 2, 2026
Merged

Modernise Packaging#110
matthewtownson merged 18 commits intoAOtools:mainfrom
matthewtownson:master

Conversation

@matthewtownson
Copy link
Copy Markdown
Member

Update CI configuration, dependencies.

  • Updated Python versions in AppVeyor and GitHub workflows to 3.10, 3.11, and 3.12.
  • Changed testing framework from nosetests to pytest.
  • Enhanced requirements with specific version constraints for numpy, scipy, matplotlib, numba, and pytest.
  • Added pyproject.toml for build system configuration.
  • Corrected README typo from PYTHONTPATH to PYTHONPATH.
  • Updated setup.py to use setuptools and improved package installation instructions.

Fix the errors in _r0_to_cn2_ and r0_to_seeing as described in  
AOtools#88
add master branch to building docs template
- Updated Python versions in AppVeyor and GitHub workflows to 3.10, 3.11, and 3.12.
- Changed testing framework from nosetests to pytest.
- Enhanced requirements with specific version constraints for numpy, scipy, matplotlib, numba, and pytest.
- Added pyproject.toml for build system configuration.
- Corrected README typo from PYTHONTPATH to PYTHONPATH.
- Updated setup.py to use setuptools and improved package installation instructions.
@matthewtownson
Copy link
Copy Markdown
Member Author

I think this is working correctly now. The issues are:

  • permissions for me to deploy docs to aotools (fair, should be fixed if PR accepted)
  • actual test failures which we should probably address.

Copilot AI review requested due to automatic review settings April 2, 2026 14:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR modernizes the project’s packaging and CI setup to target newer Python versions, migrate test execution to pytest, and tighten dependency/version constraints to better reflect a modern build and test environment.

Changes:

  • Updated CI workflows (GitHub Actions, AppVeyor, CodeQL) to newer action versions and Python 3.10–3.12 (per PR intent).
  • Switched CI test execution from nosetests to pytest and updated installation steps to pip install ..
  • Added pyproject.toml build-system config and applied minimum version constraints across requirements/install dependencies.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
test/requirements.txt Adds minimum version constraints for test dependencies.
setup.py Moves to setuptools + find_packages(), adds python_requires, and constrains runtime deps.
requirements.txt Pins minimum versions for CI-installed deps (pytest/coverage included).
README.rst Fixes PYTHONPATH typo and updates source install command to pip install ..
pyproject.toml Adds PEP 517/518 build-system configuration.
doc/source/conf.py Improves mocking for importlib behavior and updates Sphinx settings/intersphinx mapping.
appveyor.yml Updates Windows CI environment and switches to pytest-based runs.
.github/workflows/unit_test.yml Expands test matrix to 3.10–3.12 and modernizes actions + install steps.
.github/workflows/documentation.yml Modernizes actions and ensures pip is upgraded before installing docs deps.
.github/workflows/codeql-analysis.yml Updates CodeQL workflow branches and action versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.rst
Comment thread appveyor.yml
Comment thread pyproject.toml Outdated
@matthewtownson matthewtownson merged commit cbc10bb into AOtools:main Apr 2, 2026
0 of 4 checks 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