Skip to content

Consider .python-version when inferring the Python version#1790

Merged
nathanjmcdougall merged 4 commits into
mainfrom
copilot/consider-python-version-inference
Apr 4, 2026
Merged

Consider .python-version when inferring the Python version#1790
nathanjmcdougall merged 4 commits into
mainfrom
copilot/consider-python-version-inference

Conversation

Copilot AI commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Copilot AI changed the title [WIP] Fix version inference by considering .python-version Consider .python-version when inferring the Python version Apr 4, 2026
Copilot AI requested a review from nathanjmcdougall April 4, 2026 00:20
Comment thread src/usethis/_integrations/environ/python.py Outdated
Copilot AI requested a review from nathanjmcdougall April 4, 2026 00:29
@nathanjmcdougall nathanjmcdougall marked this pull request as ready for review April 4, 2026 00:52
@codecov

codecov Bot commented Apr 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codspeed-hq

codspeed-hq Bot commented Apr 4, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 2 untouched benchmarks


Comparing copilot/consider-python-version-inference (bb7c156) with main (b6feafa)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (ad8f2b7) during the generation of this report, so b6feafa was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Comment thread src/usethis/_integrations/environ/python.py
Comment thread src/usethis/_integrations/environ/python.py

Copilot AI commented Apr 4, 2026

Copy link
Copy Markdown
Contributor Author

@nathanjmcdougall I've opened a new pull request, #1802, to work on those changes. Once the pull request is ready, I'll request review from you.

@nathanjmcdougall

Copy link
Copy Markdown
Collaborator

@copilot the PR was empty and you didn't make tickets for the lessons. Try again.

…ftest

Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/fd92765f-45c3-47a9-88fc-14abf9492211

Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>

Copilot AI commented Apr 4, 2026

Copy link
Copy Markdown
Contributor Author

@copilot the PR was empty and you didn't make tickets for the lessons. Try again.

Done — changes are in this PR (bb7c156), and the lessons are recorded in #1806 (caching) and #1807 (export-functions private exclusion).

@nathanjmcdougall nathanjmcdougall merged commit d425c90 into main Apr 4, 2026
20 checks passed
@nathanjmcdougall nathanjmcdougall deleted the copilot/consider-python-version-inference branch April 4, 2026 09:09
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.

Consider .python-version when inferring the version of Python

2 participants