Skip to content

Run lint as pre-commit#948

Open
SimonIT wants to merge 1 commit intoCyberTimon:mainfrom
SimonIT:pre-commit
Open

Run lint as pre-commit#948
SimonIT wants to merge 1 commit intoCyberTimon:mainfrom
SimonIT:pre-commit

Conversation

@SimonIT
Copy link
Copy Markdown
Contributor

@SimonIT SimonIT commented Mar 31, 2026

Description

This adds a config for pre-commit so linting is run before committing. I used prek to test this.

One thing to notice is that it seems that prettier and eslint haven't been run for a while, so they will format a couple of files.
Also // '@typescript-eslint/no-explicit-any': 'off', is commented out, so eslint errors on quiet a lot of types.
Also for me clippy errors with recommending consider using sort_by_key

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Performance improvement
  • Code refactoring
  • Documentation update
  • UI/UX improvement
  • Build/CI or Dependency update

Changes Made

Screenshots/Videos

Testing

  • I have tested these changes locally and confirmed that they work as expected without issues

Test Configuration:

  • OS: (e.g. Windows 11, macOS Sonoma, Ubuntu 24.04)
  • Hardware: (e.g. Intel i7, Apple M2, Nvidia RTX 3060)

Checklist

  • My code follows the project's code style
  • I haven't added unnecessary AI-generated code comments
  • My changes generate no new warnings or errors

Additional Notes

AI Disclaimer:

Please state the involvement of AI in this PR:

  • This PR is entirely AI-generated
  • This PR is AI-generated but guided by a human
  • This PR was handwritten with AI assistance (spell check, logic suggestions, error resolving)
  • This PR contains only blood, sweat, and coffee (AI-free)

@SimonIT SimonIT requested a review from CyberTimon as a code owner March 31, 2026 14:11
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