Skip to content

chore: remove --use-pip flow and always use uv#1785

Merged
karakanb merged 2 commits intomainfrom
cleanup-remove-use-pip
Mar 11, 2026
Merged

chore: remove --use-pip flow and always use uv#1785
karakanb merged 2 commits intomainfrom
cleanup-remove-use-pip

Conversation

@karakanb
Copy link
Copy Markdown
Contributor

@karakanb karakanb commented Mar 5, 2026

Summary

  • Remove flag from run command and config path.
  • Delete legacy local/venv python executor implementations and tests.
  • Switch Python local execution to uv-only setup.
  • Move shared command runner logic into .
  • Update telemetry/docs and tests for uv-only flow.

Validation

  • make format
  • make test
  • make integration-test-light

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 5, 2026

Comments Outside Diff (1)

  1. pkg/scheduler/scheduler.go, line 128-148 (link)

    Pre-existing JSON tag mismatch was cleaned up by this removal.

    The removed field UsePip had an incorrect JSON tag json:"useUV" — the opposite of what it should be. This would have caused any persisted RunConfig JSON using the key "usePip" to be silently ignored during deserialization, while the key "useUV" would incorrectly populate a field named UsePip. Removing it entirely is the correct resolution of this data integrity bug.

Last reviewed commit: 2ca37fc

@karakanb karakanb merged commit e447ab8 into main Mar 11, 2026
6 checks passed
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.

3 participants