Skip to content

Inverse mypy and more mypy fixes#22645

Merged
hawkeye217 merged 10 commits intodevfrom
mypy
Mar 26, 2026
Merged

Inverse mypy and more mypy fixes#22645
hawkeye217 merged 10 commits intodevfrom
mypy

Conversation

@NickM-27
Copy link
Copy Markdown
Collaborator

@NickM-27 NickM-27 commented Mar 25, 2026

Please read the contributing guidelines before submitting a PR.

Proposed change

  • inverse the mypy file to specifically disable type checking everywhere that we currently get errors, making it easier to see what is left and where the gaps are
  • more mypy fixes
  • fix PR template check for dependabot

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code
  • Documentation Update

For new features

  • There is an existing feature request or discussion with community interest for this change.
    • Link:

AI disclosure

  • No AI tools were used in this PR.
  • AI tools were used in this PR. Details below:

AI tool(s) used (e.g., Claude, Copilot, ChatGPT, Cursor):

How AI was used (e.g., code generation, code review, debugging, documentation):

Extent of AI involvement (e.g., generated entire implementation, assisted with specific functions, suggested fixes):

Human oversight: Describe what manual review, testing, and validation you performed on the AI-generated portions.

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I can explain every line of code in this PR if asked.
  • UI changes including text have used i18n keys and have been added to the en locale.
  • The code has been formatted using Ruff (ruff format frigate)

@hawkeye217 hawkeye217 merged commit 0cf9d7d into dev Mar 26, 2026
8 checks passed
@hawkeye217 hawkeye217 deleted the mypy branch March 26, 2026 00:31
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