-
Notifications
You must be signed in to change notification settings - Fork 4
Modernize #134
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
Modernize #134
Conversation
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
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 project's build and documentation tooling by migrating from legacy configurations to modern Python packaging standards. The changes streamline the development workflow and documentation generation.
Changes:
- Migrated from
[project.optional-dependencies]to PEP 735[dependency-groups]format and replacedpre-commitwithprek - Consolidated GitHub Actions workflows into streamlined CI/CD pipelines with uv package manager
- Restructured Sphinx documentation to use a flat structure instead of nested
source/directory
Reviewed changes
Copilot reviewed 9 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Updated dependency groups to use PEP 735 format and replaced pre-commit with prek |
| docs/make.bat | Removed legacy Windows Makefile for Sphinx (no longer needed) |
| docs/Makefile | Removed legacy Unix Makefile for Sphinx (no longer needed) |
| docs/index.rst | Added new documentation index with module references and navigation |
| docs/history.rst | Added project history documentation |
| docs/conf.py | Added Sphinx configuration with autoapi and releases extensions |
| docs/changelog.rst | Added changelog documentation using releases format |
| docs/.readthedocs.yaml | Updated Sphinx configuration path to flat structure |
| .pre-commit-config.yaml | Updated pre-commit-hooks to v6.0.0 and removed devcontainer exclusion |
| .github/workflows/python-ci.yaml | Consolidated CI workflow to use uv with single lint-test job |
| .github/workflows/docs.yaml | Added dedicated docs workflow for building and deploying to GitHub Pages |
| .github/workflows/dependency-review.yaml | Removed quotes from workflow name and step names |
| .github/dependabot.yaml | Removed devcontainers ecosystem and changed pip to uv |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
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
Copilot reviewed 10 out of 15 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
pyproject.toml:45
- The Ruff target-version is set to "py39" but the project now requires Python >=3.10. This should be updated to "py310" to ensure Ruff checks align with the minimum supported Python version.
target-version = "py39"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
CodSpeed Performance ReportMerging this PR will not alter performanceComparing Summary
|
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Signed-off-by: GitHub <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
Copilot reviewed 12 out of 16 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
pyproject.toml:45
- The ruff target-version should be updated to 'py310' to match the updated requires-python constraint of '>=3.10'.
target-version = "py39"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.