Skip to content

chore: enable default pyrefly checking - #151

Merged
ruben-arts merged 1 commit into
RoboStack:masterfrom
baszalmstra:chore/enable-default-pyrefly-checking
Sep 4, 2026
Merged

chore: enable default pyrefly checking#151
ruben-arts merged 1 commit into
RoboStack:masterfrom
baszalmstra:chore/enable-default-pyrefly-checking

Conversation

@baszalmstra

@baszalmstrabaszalmstra commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Pyrefly was falling back to its basic preset, so CI only caught parse errors, missing imports, and a small set of local issues. This moves package code to the default preset and makes the check part of the Pixi and CI workflows.

The default preset exposed incomplete type information around generated YAML documents and nullable command-line state. Those are now made explicit without changing the generated recipe format. While fixing the diagnostics, the migration version parser was also corrected to search the version segment rather than the accumulating integer list.

The type check deliberately excludes test_*.py: those modules use partial fakes and mocks that are covered by pytest rather than treated as production interfaces.

Testing

  • pixi run typecheck --output-format min-text --progress-bar no
  • pixi run lint-check && pixi run fmt-check
  • pixi run python -m pytest vinca/test_archive_sources.py vinca/test_distro_dependencies.py vinca/test_naming_scheme.py vinca/test_provenance.py vinca/test_snapshot_metadata.py vinca/test_generate_gha.py vinca/test_pinning.py vinca/test_v1_selectors.py vinca/test_mutex.py vinca/test_sources.py vinca/test_recipes.py — 166 passed

@baszalmstra
baszalmstraforce-pushed the chore/enable-default-pyrefly-checking branch from 576f69a to dbc5030CompareSeptember 4, 2026 12:43
@baszalmstra
baszalmstra marked this pull request as ready for review September 4, 2026 12:45
@ruben-arts
ruben-arts merged commit 34abd77 into RoboStack:masterSep 4, 2026
3 checks passed
Sign up for freeto 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

@baszalmstra@ruben-arts