Skip to content

Auto-fix pre-commit errors#480

Merged
MattyTheHacker merged 13 commits intomainfrom
pre-commit-ci
May 4, 2025
Merged

Auto-fix pre-commit errors#480
MattyTheHacker merged 13 commits intomainfrom
pre-commit-ci

Conversation

@MattyTheHacker
Copy link
Copy Markdown
Member

@MattyTheHacker MattyTheHacker commented May 3, 2025

Adds a workflow to automatically fix fixable issues on pre-commit, commit message can be configured if desired.
https://pre-commit.ci/lite.html
https://github.com/pre-commit-ci/lite-action

Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
@MattyTheHacker MattyTheHacker requested a review from Copilot May 3, 2025 13:59
@MattyTheHacker MattyTheHacker self-assigned this May 3, 2025
Copy link
Copy Markdown
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 aims to test automatic CI fixes by adding a new job in the GitHub workflow that runs pre-commit checks before the main build and deployment steps.

  • Introduces a new job "auto-pre-commit" with steps for checking out code, setting up Python, and performing pre-commit checks.
  • Updates the "uv-check" job to depend on the successful execution of the "auto-pre-commit" job.

Comment thread .github/workflows/check-build-deploy.yaml Outdated
MattyTheHacker and others added 3 commits May 3, 2025 15:00
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
MattyTheHacker and others added 4 commits May 3, 2025 15:09
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
@MattyTheHacker MattyTheHacker marked this pull request as ready for review May 3, 2025 14:30
@MattyTheHacker MattyTheHacker enabled auto-merge (squash) May 3, 2025 14:31
@MattyTheHacker MattyTheHacker changed the title Test auto CI fixes Auto-fix pre-commit fails May 3, 2025
Comment thread .github/workflows/check-build-deploy.yaml
Comment thread .github/workflows/check-build-deploy.yaml Outdated
@MattyTheHacker MattyTheHacker added enhancement New feature or request deployment Changes to the deployment or CI/CD configuration labels May 4, 2025
@CarrotManMatt CarrotManMatt removed the enhancement New feature or request label May 4, 2025
@CarrotManMatt CarrotManMatt changed the title Auto-fix pre-commit fails Auto-fix pre-commit errors May 4, 2025
@MattyTheHacker MattyTheHacker merged commit f5aadc1 into main May 4, 2025
9 checks passed
@MattyTheHacker MattyTheHacker deleted the pre-commit-ci branch May 4, 2025 07:20
MattyTheHacker added a commit that referenced this pull request May 4, 2025
Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployment Changes to the deployment or CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants