Skip to content

Configure Python Env Tools #1866

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 330 commits into
base: main
Choose a base branch
from

Conversation

DonJayamanne
Copy link
Owner

No description provided.

karthiknadig and others added 30 commits September 6, 2024 10:48
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
Reverts #23977

Have to revert #23977
with issue: #23743 due
to #24069

Will revisit why #23743
is breaking if contained inside other top level (in ast term) code
block, and look into how to support
#23743 without
breaking.
Resolves: #23929 

TODO: (debt --> in separate PR) Have ensureTerminal return
Promise<Terminal> instead of Promise<void> and saving this in the
TerminalService class. Would avoid many uses of the !, and maybe even
get to throw away the TerminalService class itself.
Addresses #24068

- Enable proposed APIs for `codeActionAI`
- Pin the engine to `1.94.0-20240913`
Resolves: #3828
Breaking #23953 down into
two PR
1. problem matcher --> make sure to cover case where there is invalid
strings printed before the Error (e.g. NameError or ValueError)
2. Whether we will replace 'Run In Terminal by contributing task with
the problem matcher attached.

---------

Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
`enter` should apply to the inline chat widget if it is focused, not the
parent code editor
both the repl editor and the IW can be accounted for with the new
compositeNotebook context key
…ollection (#24111)

Resolves: #23930
- setting to opt out of PYTHONSTARTUP injection.

---------

Co-authored-by: Courtney Webster <60238438+cwebster-99@users.noreply.github.com>
Need logging to further investigate
#24123 (comment)
for windows.
From discussion in the original issue:
#23930 (comment)
Making default to be false for September stable, perhaps we could turn
it on to true for insiders AFTER once we ship out stable.
/cc @Tyriar
To better control flow of:
#24141
Adding experimental tag:
This way, we are able to control the default value without having to
recovery release in the worst case scenario.
…n.enabled (#24153)

Resolves: #24143 
/cc @cwebster-99 @Tyriar

---------

Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
Bumps [importlib-metadata](https://github.com/python/importlib_metadata)
from 8.4.0 to 8.5.0.
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/python/importlib_metadata/blob/main/NEWS.rst">importlib-metadata's
changelog</a>.</em></p>
<blockquote>
<h1>v8.5.0</h1>
<h2>Features</h2>
<ul>
<li>Deferred import of zipfile.Path (<a
href="https://redirect.github.com/python/importlib_metadata/issues/502">#502</a>)</li>
<li>Deferred import of json (<a
href="https://redirect.github.com/python/importlib_metadata/issues/503">#503</a>)</li>
<li>Rely on zipp overlay for zipfile.Path.</li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/python/importlib_metadata/commit/b34810b1e0665580a91ea19b6317a1890ecd42c1"><code>b34810b</code></a>
Finalize</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/8c1d1fa4f4e2160fef17b0bb7fef2ca276b53a99"><code>8c1d1fa</code></a>
Merge pull request <a
href="https://redirect.github.com/python/importlib_metadata/issues/501">#501</a>
from Avasam/Pass-mypy-and-link-issues</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/afa39e8e08b48fbedd3b8ac94cf58de39ff09c35"><code>afa39e8</code></a>
Back out changes to tests._path</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/8b909f9b1cdfc056eff3acd38337661ab24ef8b3"><code>8b909f9</code></a>
Merge pull request <a
href="https://redirect.github.com/python/importlib_metadata/issues/503">#503</a>
from danielhollas/defer-json</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/2a3f50d8bbd41fc831676e7dc89d84c605c85760"><code>2a3f50d</code></a>
Add news fragment.</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/3f78dc17786e0e0290db450e843ac494af0158e9"><code>3f78dc1</code></a>
Add comment to protect the deferred import.</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/18eb2da0ee267394c1735bec5b1d9f2b0fa77dd9"><code>18eb2da</code></a>
Revert &quot;Defer platform import&quot;</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/58832f234ba1aea5906c4d56f4b2c4a21d640608"><code>58832f2</code></a>
Merge pull request <a
href="https://redirect.github.com/python/importlib_metadata/issues/502">#502</a>
from danielhollas/defer-zipp</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/e3ce33b45e572824b482049570cac13da543999b"><code>e3ce33b</code></a>
Add news fragment.</li>
<li><a
href="https://github.com/python/importlib_metadata/commit/d11b67fed9f21503ca369e33c917a8038994ce0b"><code>d11b67f</code></a>
Add comment to protect the deferred import.</li>
<li>Additional commits viewable in <a
href="https://github.com/python/importlib_metadata/compare/v8.4.0...v8.5.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=importlib-metadata&package-manager=pip&previous-version=8.4.0&new-version=8.5.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Removed redundant @typescript-eslint/no-explicit-any suppression.

Resolves #24076
Since setting name is changed:
#24153
Adjusting the test accordingly.
…24164)

Reference: #23773 and
#23773 (comment)

It seems that pixi warnings may be confusing unnecessary confusion among
folks who may not even intend to use pixi environment. Changing log
level for clarity and further help diagnosing problems that may be
unrelated to pixi.
…#24180)

Removing branch coverage from the payload as the initial way it was
being discovered / added was not documented / in the coverage.py API and
therefore is not guaranteed to be stable. Removing for now with plans to
add upstream and re-include in the extension here

coverage PR: #24118
eleanorjboyd and others added 17 commits May 2, 2025 12:14
…#25037)

Resolve: #25013 

Python shell integration env var injection via env var collection was
getting cleared undesirable, when user had opted out of terminal env var
experiment.

We want to inject PYTHONSTARTUP regardless of the experiment, depending
on user setting.
Add shortTitle to execSelectionInTerminal command
This fixes #25016

Added some changes to package.json, so that the extension gets activated
whenever there is a file with the name `pylock.toml` or match the
regular expression `r"^pylock\.([^.]+)\.toml$"`.

I followed [PEP 751](https://peps.python.org/pep-0751/#file-name)'s
naming specification.

![Screenshot
(127)](https://github.com/user-attachments/assets/476abd9b-9251-457b-bdcc-ae3d3c16fd73)
Required to use `vscode.lm.registerTool()`
* Tested with .venv, global conda, local conda and global Python
* Tested both tools

![Screenshot 2025-05-14 at 11 54
44](https://github.com/user-attachments/assets/dab561f7-a66d-4157-8e04-86be433fb0eb)
![Screenshot 2025-05-14 at 11 57
39](https://github.com/user-attachments/assets/ce727718-56fb-4948-9764-bc8df3dda565)
![Screenshot 2025-05-14 at 11 57
49](https://github.com/user-attachments/assets/f1a5c643-b079-4306-be89-654a2043e8b4)
@DonJayamanne DonJayamanne force-pushed the don/injured-frog branch 2 times, most recently from 0c5521c to abdf539 Compare May 20, 2025 00:52
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.