Skip to content

Conversation

@ChristopherLi05
Copy link
Member

Full addresses #4

@ChristopherLi05
Copy link
Member Author

Also addresses #1

@crypto-a crypto-a requested a review from Copilot September 16, 2025 14:18
Copy link
Contributor

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 introduces linting infrastructure to the project by adding pre-commit hooks and GitHub Actions workflow for code quality enforcement.

  • Added pre-commit configuration with Ruff linter and standard hooks
  • Created GitHub Actions workflow to run linting checks on pull requests
  • Modified existing test workflow to only run on pull requests (commented out push triggers)

Reviewed Changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.pre-commit-config.yaml New pre-commit configuration with Ruff linter and standard code quality hooks
.github/workflows/lint-project.yml New GitHub Actions workflow to run pre-commit checks on pull requests
pyproject.toml Added pre-commit as development dependency
tests/test_import.py Added noqa comment to suppress linter warning
.github/workflows/test-project.yml Commented out push trigger to only run tests on pull requests

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@crypto-a crypto-a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@crypto-a crypto-a merged commit 3af3e27 into main Sep 16, 2025
16 checks passed
@crypto-a crypto-a deleted the 4/lint branch September 16, 2025 14:32
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.

3 participants