Skip to content

Conversation

@iraedeus
Copy link
Collaborator

No description provided.

@iraedeus iraedeus requested a review from Copilot July 17, 2025 17:39

This comment was marked as outdated.

@iraedeus iraedeus force-pushed the iraedeus/setup-precommit branch from 75da959 to a042d2f Compare July 17, 2025 19:01
@iraedeus iraedeus requested a review from Copilot July 17, 2025 19:24
Copy link

Copilot AI left a 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 sets up pre-commit hooks for the project to automate code quality checks before commits. The changes include adding pre-commit configuration, updating CI workflows, and applying automatic formatting fixes across test files.

  • Adds pre-commit configuration with hooks for code formatting (ruff), linting, type checking (mypy), and basic file checks
  • Updates GitHub Actions workflow to use pre-commit instead of individual tool runs
  • Applies automatic import sorting and whitespace formatting fixes across multiple test files

Reviewed Changes

Copilot reviewed 35 out of 38 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.pre-commit-config.yaml Adds pre-commit configuration with ruff, mypy, and basic file validation hooks
.github/workflows/check.yml Updates CI workflow to use pre-commit and newer action versions
pyproject.toml Updates ruff configuration and adds type stub dependencies
CONTRIBUTING.md Adds mandatory pre-commit setup instructions for contributors
Multiple test files Applies automatic import sorting and whitespace cleanup
docs/source/index.rst Removes trailing whitespace

@iraedeus iraedeus merged commit b0e8e20 into PySATL:main Jul 18, 2025
3 checks passed
@iraedeus iraedeus deleted the iraedeus/setup-precommit branch July 18, 2025 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant