-
Notifications
You must be signed in to change notification settings - Fork 1
Melf/update ci 2025 #93
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 modernizes the CI/CD infrastructure and project configuration by migrating from setup.py to pyproject.toml, updating GitHub workflows to use reusable workflows from pytket-workflows, and switching from black/pylint to ruff for formatting and linting. The changes also update repository references from CQCL to Quantinuum and add Python 3.13 support.
Key Changes
- Migrated from
setup.pytopyproject.tomlwith modern packaging configuration - Replaced custom GitHub Actions workflows with reusable workflows from
Quantinuum/pytket-workflows - Switched from
black/pylinttorufffor code formatting and linting
Reviewed changes
Copilot reviewed 27 out of 29 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | New project configuration file with dependencies and metadata |
| setup.py | Removed legacy setup.py in favor of pyproject.toml |
| .github/workflows/all-checks.yml | New workflow using reusable static and unit test workflows |
| .github/workflows/release.yml | New workflow using reusable release workflow |
| .github/workflows/docs.yml | New workflow using reusable documentation workflow |
| .github/workflows/build_and_test.yml | Removed custom build and test workflow |
| .github/workflows/lint.yml | Removed custom lint workflow |
| .github/workflows/build-test | Removed custom build-test script |
| .github/workflows/fetch-modules | Removed unused fetch-modules script |
| ruff.toml | Updated exclusions and removed TODO comments |
| .pre-commit-config.yaml | Added pre-commit configuration for automated checks |
| .pylintrc | Removed pylint configuration in favor of ruff |
| mypy-check | Removed custom mypy check script |
| tests/test-requirements.txt | Removed test requirements (moved to pyproject.toml) |
| tests/test_stim.py | Applied ruff formatting fixes |
| pytket/extensions/stim/backends/stim_backend.py | Reordered imports per ruff standards |
| pytket/extensions/stim/_metadata.py | Added metadata file to package |
| _metadata.py | Removed root metadata file |
| MANIFEST.in | Updated to reference new metadata location |
| README.md | Updated documentation with new development instructions and repository links |
| docs/README.md | Updated repository references from CQCL to Quantinuum |
| docs/index.rst | Updated issue tracker link |
| docs/pytket-docs-theming | Updated submodule commit reference |
| .gitmodules | Updated submodule URL from CQCL to Quantinuum |
| docs/install.sh | Removed deprecated install script |
| docs/build-docs.sh | Removed deprecated build script |
| pytest.ini | Added pytest configuration for warning filters |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
0bbb969 to
4f935c6
Compare
Description
Please summarise the changes.
Related issues
Please mention any github issues addressed by this PR.
Checklist