Skip to content

fix: resolve follow-up quality findings#1222

Merged
mldangelo-oai merged 1 commit intomainfrom
mdangelo/codex/fix-quality-findings-followup
May 3, 2026
Merged

fix: resolve follow-up quality findings#1222
mldangelo-oai merged 1 commit intomainfrom
mdangelo/codex/fix-quality-findings-followup

Conversation

@mldangelo-oai
Copy link
Copy Markdown
Contributor

Summary

  • centralize the restricted Python-version list used by the reduced CI lane
  • replace the progress callback function-with-attached-attributes fixture with an explicit recorder object
  • document and guard the internal pytest temp-path factory access used by xdist setup
  • replace the protocol ellipsis body that CodeQL flagged as a no-op statement

Validation

  • UV_CACHE_DIR=/tmp/modelaudit-uv-cache uv run ruff format modelaudit/ packages/modelaudit-picklescan/src packages/modelaudit-picklescan/tests tests/
  • UV_CACHE_DIR=/tmp/modelaudit-uv-cache uv run ruff check --fix modelaudit/ packages/modelaudit-picklescan/src packages/modelaudit-picklescan/tests tests/
  • UV_CACHE_DIR=/tmp/modelaudit-uv-cache uv run mypy modelaudit/ packages/modelaudit-picklescan/src packages/modelaudit-picklescan/tests tests/
  • PROMPTFOO_DISABLE_TELEMETRY=1 UV_CACHE_DIR=/tmp/modelaudit-uv-cache uv run pytest -n auto -m "not slow and not integration" --maxfail=1

@mldangelo-oai mldangelo-oai merged commit 2968961 into main May 3, 2026
29 checks passed
@mldangelo-oai mldangelo-oai deleted the mdangelo/codex/fix-quality-findings-followup branch May 3, 2026 03: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.

1 participant