-
Notifications
You must be signed in to change notification settings - Fork 5
Advanced next into qa #218
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
Conversation
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.1 → v0.14.2](astral-sh/ruff-pre-commit@v0.14.1...v0.14.2) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.2 → v0.14.4](astral-sh/ruff-pre-commit@v0.14.2...v0.14.4) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Bumps [prefix-dev/setup-pixi](https://github.com/prefix-dev/setup-pixi) from 0.9.2 to 0.9.3. - [Release notes](https://github.com/prefix-dev/setup-pixi/releases) - [Commits](prefix-dev/setup-pixi@v0.9.2...v0.9.3) --- updated-dependencies: - dependency-name: prefix-dev/setup-pixi dependency-version: 0.9.3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...v6) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.4 → v0.14.7](astral-sh/ruff-pre-commit@v0.14.4...v0.14.7) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.7 → v0.14.8](astral-sh/ruff-pre-commit@v0.14.7...v0.14.8) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Bumps [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request) from 7 to 8. - [Release notes](https://github.com/peter-evans/create-pull-request/releases) - [Commits](peter-evans/create-pull-request@v7...v8) --- updated-dependencies: - dependency-name: peter-evans/create-pull-request dependency-version: '8' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.8 → v0.14.9](astral-sh/ruff-pre-commit@v0.14.8...v0.14.9) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.9 → v0.14.10](astral-sh/ruff-pre-commit@v0.14.9...v0.14.10) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.10 → v0.14.11](astral-sh/ruff-pre-commit@v0.14.10...v0.14.11) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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 merges the "advanced next" branch into QA, primarily consisting of comprehensive documentation restructuring and updates to support infrastructure. The changes move away from legacy documentation to a modern structure while adding deprecation warnings for the NOVA backend.
Changes:
- Removed legacy documentation files and added new comprehensive user guides and API reference documentation
- Added warnings throughout the codebase indicating NOVA backend support is currently paused
- Updated GitHub Actions workflow configurations and pre-commit hooks to newer versions
- Added documentation build requirements and Read the Docs configuration
Reviewed changes
Copilot reviewed 131 out of 139 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
tests/data/pleiades_data |
Updated subproject commit reference |
src/pleiades/sammy/factory.py |
Added runtime warning for NOVA backend being paused |
src/pleiades/sammy/config.py |
Added docstring warning about NOVA backend status |
src/pleiades/sammy/backends/nova_ornl.py |
Added comprehensive module-level warning about NOVA backend pause |
legacy/docs/* |
Removed all legacy documentation files |
docs/* |
Added new comprehensive documentation structure with guides and API references |
.readthedocs.yaml |
Enabled pip installation in documentation build |
.pre-commit-config.yaml |
Updated ruff version from v0.14.1 to v0.14.11 |
.github/workflows/* |
Updated GitHub Actions versions |
Comments suppressed due to low confidence (2)
docs/quickstart.rst:1
- Corrected spelling of 'recieve' to 'receive'.
src/pleiades/sammy/backends/nova_ornl.py:1 - The status date 'January 2025' is now over a year old (current date is January 14, 2026). Consider updating this documentation to reflect the current status or adding a more recent timestamp if the pause is ongoing.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.