Skip to content

style: add pre-commit for style#101

Merged
frode-aarstad merged 1 commit intomainfrom
add-pre-commit-for-style
Dec 1, 2025
Merged

style: add pre-commit for style#101
frode-aarstad merged 1 commit intomainfrom
add-pre-commit-for-style

Conversation

@frode-aarstad
Copy link
Copy Markdown
Contributor

Add pre-commit style to help maintain code quality and some standards

@frode-aarstad frode-aarstad self-assigned this Nov 5, 2025
@frode-aarstad frode-aarstad added the enhancement New feature or request label Nov 5, 2025
@frode-aarstad frode-aarstad force-pushed the add-pre-commit-for-style branch 3 times, most recently from b0a6ba3 to 16f79e3 Compare November 5, 2025 13:46
@frode-aarstad frode-aarstad changed the title Add pre-commit for style style: add pre-commit for style Nov 6, 2025
@frode-aarstad frode-aarstad force-pushed the add-pre-commit-for-style branch 4 times, most recently from 777e98a to d9e35dc Compare November 28, 2025 08:36
Copy link
Copy Markdown

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 pre-commit hooks to enforce code style standards across the repository. The changes include:

  • Addition of pre-commit configuration with ruff, trailing whitespace checks, and other formatting tools
  • Style dependency group in pyproject.toml
  • Automated style checking workflow in GitHub Actions
  • Comprehensive code formatting updates across Python files (trailing whitespace removal, line length fixes, quote normalization)
  • Documentation and configuration file formatting improvements

Reviewed changes

Copilot reviewed 16 out of 28 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.pre-commit-config.yaml New pre-commit configuration with hooks for formatting, linting, and dependency locking
.github/workflows/style.yml New CI workflow to enforce style checks on PRs
pyproject.toml Added style optional dependency group for pre-commit
uv.lock Updated dependencies including pre-commit and its dependencies
tests/**/*.py Applied automated formatting (trailing whitespace, line length, imports)
data/scripts/*.py Removed unused variables and improved formatting
README.md Added documentation for style requirements and pre-commit usage
Various config files Applied formatting fixes (trailing whitespace, quotes)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@roliveira roliveira left a comment

Choose a reason for hiding this comment

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

Thanks!

@frode-aarstad frode-aarstad merged commit 2602c8c into main Dec 1, 2025
3 checks passed
@frode-aarstad frode-aarstad deleted the add-pre-commit-for-style branch December 1, 2025 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants