Skip to content

Resolve Ruff safety findings in the PyQt task model - #184

Merged
ZFordDev merged 2 commits into
mainfrom
codex/180-pyqt-model-safety
Aug 27, 2026
Merged

Resolve Ruff safety findings in the PyQt task model#184
ZFordDev merged 2 commits into
mainfrom
codex/180-pyqt-model-safety

Conversation

@ZFordDev

Copy link
Copy Markdown
Owner

Closes #180

This PR replaces call expressions in Qt model defaults with a shared invalid QModelIndex, explicitly marks the sort map as class-owned state, and adds behavior coverage for root and child model indexes.

This is the final PR in the Ruff remediation stack and is stacked on #183.

Validation:

  • Ruff check passes across src and tests with zero findings.
  • Ruff formatting passes for all 79 files.
  • The complete source and test trees compile.
  • The PyQt behavior test is committed for GitHub Actions; PyQt6 is not installed in the local bundled runtime.
  • Local mypy is unavailable in the bundled runtime, so CI will run it after installing declared dev dependencies.

@ZFordDev
ZFordDev force-pushed the codex/180-pyqt-model-safety branch from 3ddfcc3 to a83f2bc Compare August 27, 2026 18:41
Base automatically changed from codex/179-exception-boundaries to main August 27, 2026 18:54
@ZFordDev
ZFordDev merged commit a50e303 into main Aug 27, 2026
2 of 3 checks passed
@ZFordDev ZFordDev linked an issue Aug 27, 2026 that may be closed by this pull request
@ZFordDev
ZFordDev deleted the codex/180-pyqt-model-safety branch August 27, 2026 18:54
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.

Resolve Ruff safety findings in the PyQt task model

1 participant