Skip to content

Clarify uv-managed Python requirement in local setup docs - #622

Draft
jbampton with Copilot wants to merge 1 commit into
mainfrom
copilot/clarify-readme
Draft

jbampton with Copilot wants to merge 1 commit into
mainfrom
copilot/clarify-readme

Conversation

Copilot AI commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

The README treated Python as an initial prerequisite for local development, even though uv can provision a compatible interpreter for the repository’s Python-based dev tooling. This updates the setup guidance so contributors do not assume a preinstalled Python is required when using uv.

  • Prerequisites

    • Rewords the Python requirement as optional for contributors who want to supply their own interpreter.
    • Clarifies that uv can download a compatible Python version when needed.
  • prek setup guidance

    • Adds an explicit note in the local quality-check section that the documented uv workflow works even when Python 3.13+ is not already installed locally.
  • Example

    # Install Python dev tools, including prek
    uv sync --only-group dev
    
    # Run the standard hooks across the repository
    uv run prek run --all-files

Co-authored-by: jbampton <418747+jbampton@users.noreply.github.com>
@deepsource-io

deepsource-io Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 6e080c5...27a8107 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Sep 26, 2026 1:47a.m. Review ↗
Secrets Sep 26, 2026 1:47a.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants