Add pre commit run to ci - #8
Merged
Merged
Conversation
added 2 commits
October 7, 2025 00:23
…upgrade idempotency, pre-commit reliability, project URLs in metadata, and README installation notes. - T011: Added snapshots for CLI help commands to catch accidental UX changes. - T012: Implemented tests to ensure `upgrade-repository` is safe to re-run. - T013: Improved pre-commit reliability by addressing Node bootstrap issues and documenting setup. - T014: Added project URLs to `pyproject.toml` for better package metadata. - T015: Updated README with installation notes for Homebrew and apt, and clarified graph limitations. - Updated `uv.lock` to include new dependencies: cfgv, identify, nodeenv, pre-commit, and pyyaml.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds pre-commit integration to the CI pipeline and updates project documentation formatting. The main goal is to improve code quality automation by running pre-commit hooks in CI while also standardizing markdown formatting across ticket files.
- Added a new CI job that runs pre-commit hooks across multiple Python versions
- Consolidated linting and type checking into the pre-commit workflow instead of separate CI steps
- Updated markdown formatting in ticket files to improve readability with proper line breaks
Reviewed Changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/ci.yml |
Added lint-and-typecheck job that runs pre-commit hooks, removed separate lint/typecheck steps from build-test job |
pyproject.toml |
Added pre-commit dependency group and ruff ignore rules for self/cls annotations |
.markdownlint-cli2.yaml |
New configuration file for markdownlint with disabled rules for emphasis and consecutive blank lines |
.github/copilot-instructions.md |
Reformatted with improved line breaks for better readability |
tickets/*.md |
Applied consistent markdown formatting with proper line breaks and spacing across all ticket files |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.