Skip to content

Add pre-commit configuration for code quality checks - #26

Merged
johnny603 merged 1 commit into
mainfrom
johnny603-patch-1-1
Sep 3, 2026
Merged

Add pre-commit configuration for code quality checks#26
johnny603 merged 1 commit into
mainfrom
johnny603-patch-1-1

Conversation

@johnny603

Copy link
Copy Markdown
Owner

Summary

Describe the changes introduced by this pull request.

Type of Change

Select all that apply:

  • feat: New feature
  • fix: Bug fix
  • docs: Documentation changes
  • test: New or updated tests
  • refactor: Code restructuring without behavior changes
  • chore: Maintenance, tooling, CI/CD, dependencies
  • ci: GitHub Actions or automation changes

Related Issue

Closes #

Testing

Describe how you tested your changes.

pytest
ruff check .
bandit -r .

Checklist

  • My commit messages follow the Conventional Commits specification.
  • I have added or updated tests where appropriate.
  • I have updated documentation where appropriate.
  • All CI checks pass.
  • My changes do not introduce known security issues.

Example Commit Messages

feat: add Docker-based puzzle validation
fix: correct level submission response handling
docs: update installation instructions
test: add health endpoint smoke tests
refactor: simplify puzzle lookup logic
chore: upgrade Flask dependency
ci: add CodeQL workflow

Screenshots / Logs (Optional)

Attach screenshots or logs if relevant.

@johnny603
johnny603 requested a lite review from Copilot September 3, 2026 07:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@johnny603
johnny603 merged commit 47e7a68 into main Sep 3, 2026
9 checks passed
@johnny603
johnny603 deleted the johnny603-patch-1-1 branch September 3, 2026 07: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.

2 participants