Skip to content

Enable configuring which python to use in more locations#685

Merged
keyboardDrummer merged 2 commits intomainfrom
configurePythonCmd
Mar 27, 2026
Merged

Enable configuring which python to use in more locations#685
keyboardDrummer merged 2 commits intomainfrom
configurePythonCmd

Conversation

@keyboardDrummer
Copy link
Copy Markdown
Contributor

@keyboardDrummer keyboardDrummer commented Mar 27, 2026

Changes

This change enables me to use the Miso installed Python on my local machine, so I can run StrataTest/Languages/Python/AnalyzeLaurelTest.lean. With the system Python and the Python brew installation I was having issues.

Testing

  • Was locally able to run StrataTest/Languages/Python/AnalyzeLaurelTest.lean

@keyboardDrummer keyboardDrummer marked this pull request as ready for review March 27, 2026 13:12
@keyboardDrummer keyboardDrummer requested a review from a team March 27, 2026 13:12
Copy link
Copy Markdown
Contributor

@joehendrix joehendrix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't test locally, but am familiar with the code, and these all seem like good changes that address inconsistencies in current python paths.

@keyboardDrummer keyboardDrummer added this pull request to the merge queue Mar 27, 2026
Merged via the queue into main with commit 5b7eb62 Mar 27, 2026
15 checks passed
@keyboardDrummer keyboardDrummer deleted the configurePythonCmd branch March 27, 2026 19:25
olivier-aws pushed a commit that referenced this pull request Mar 30, 2026
### Changes
This change enables me to use the Miso installed Python on my local
machine, so I can run
`StrataTest/Languages/Python/AnalyzeLaurelTest.lean`. With the system
Python and the Python brew installation I was having issues.

### Testing
- Was locally able to run
`StrataTest/Languages/Python/AnalyzeLaurelTest.lean`
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