-
Notifications
You must be signed in to change notification settings - Fork 2
Melf/update ci 2025 #95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 updates the project's CI/CD infrastructure and modernizes the build system by migrating from setup.py to pyproject.toml and consolidating tooling configurations. The changes include updating repository references from CQCL to Quantinuum, adopting uv as the package manager, and replacing custom scripts with reusable workflows.
Key changes:
- Migration from
setup.pytopyproject.tomlfor build configuration - Replacement of custom CI workflows with Quantinuum's shared workflow templates
- Update of all repository URLs from CQCL to Quantinuum organization
Reviewed changes
Copilot reviewed 27 out of 28 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | New build configuration file defining project metadata, dependencies, and dev tools |
| setup.py | Removed legacy setup.py build configuration |
| pytket/extensions/pyzx/_metadata.py | Moved metadata from root to package directory |
| _metadata.py | Removed root-level metadata file |
| .github/workflows/all-checks.yml | New workflow using shared static and unit test workflows |
| .github/workflows/build_and_test.yml | Removed legacy custom build/test workflow |
| .github/workflows/lint.yml | Removed standalone lint workflow (now in shared workflows) |
| .github/workflows/release.yml | New release workflow using shared release workflow |
| .github/workflows/docs.yml | New docs workflow using shared documentation workflow |
| .github/workflows/build-test | Removed legacy bash script for building and testing |
| .github/workflows/fetch-modules | Removed legacy module fetching script |
| tests/test-requirements.txt | Removed test requirements (moved to pyproject.toml) |
| mypy-check | Removed legacy mypy check script |
| .pylintrc | Removed pylint configuration (replaced by ruff) |
| .pre-commit-config.yaml | New pre-commit configuration with ruff, prettier, and other hooks |
| ruff.toml | Updated exclusions and removed TODO comments from rule configurations |
| README.md | Updated documentation to reflect new tooling, Python version support, and repository URLs |
| docs/README.md | Updated repository URLs from CQCL to Quantinuum |
| docs/index.rst | Updated issue tracker URL to Quantinuum organization |
| docs/build-docs.sh | Removed legacy docs build script |
| docs/install.sh | Removed legacy docs installation script |
| docs/pytket-docs-theming | Updated submodule commit reference |
| .gitmodules | Updated submodule URL to Quantinuum organization |
| pytket/extensions/pyzx/pyzx_convert.py | Minor code cleanup: removed blank line and simplified dict comprehension |
| tests/pyzx_convert_test.py | Reordered imports to group standard library, external packages, and local modules |
| MANIFEST.in | Updated metadata file path to new location |
| pytest.ini | New pytest configuration with filterwarnings |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
a5b67a6 to
46c507b
Compare
Description
Please summarise the changes.
Related issues
Please mention any github issues addressed by this PR.
Checklist