Skip to content
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

Pytest Report fails due to AttributeError #21784

Closed
johnmarston-nhs opened this issue Aug 8, 2023 · 10 comments · Fixed by #21859
Closed

Pytest Report fails due to AttributeError #21784

johnmarston-nhs opened this issue Aug 8, 2023 · 10 comments · Fixed by #21859
Assignees
Labels
area-testing bug Issue identified by VS Code Team member as probable bug verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@johnmarston-nhs
Copy link

johnmarston-nhs commented Aug 8, 2023

Type: Bug

Behaviour

Expected vs. Actual

Expected behaviour:
When a test fails, the pytest report should be printed to the terminal.

Actual:
INTERNALERROR is reported to the terminal, along with a stacktrace as follows:

INTERNALERROR> Traceback (most recent call last):
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/_pytest/main.py", line 269, in wrap_session
INTERNALERROR>     session.exitstatus = doit(config, session) or 0
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/_pytest/main.py", line 323, in _main
INTERNALERROR>     config.hook.pytest_runtestloop(session=session)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_hooks.py", line 265, in __call__
INTERNALERROR>     return self._hookexec(self.name, self.get_hookimpls(), kwargs, firstresult)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_manager.py", line 80, in _hookexec
INTERNALERROR>     return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_callers.py", line 60, in _multicall
INTERNALERROR>     return outcome.get_result()
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_result.py", line 60, in get_result
INTERNALERROR>     raise ex[1].with_traceback(ex[2])
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_callers.py", line 39, in _multicall
INTERNALERROR>     res = hook_impl.function(*args)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/_pytest/main.py", line 348, in pytest_runtestloop
INTERNALERROR>     item.config.hook.pytest_runtest_protocol(item=item, nextitem=nextitem)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_hooks.py", line 265, in __call__
INTERNALERROR>     return self._hookexec(self.name, self.get_hookimpls(), kwargs, firstresult)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_manager.py", line 80, in _hookexec
INTERNALERROR>     return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_callers.py", line 60, in _multicall
INTERNALERROR>     return outcome.get_result()
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_result.py", line 60, in get_result
INTERNALERROR>     raise ex[1].with_traceback(ex[2])
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_callers.py", line 39, in _multicall
INTERNALERROR>     res = hook_impl.function(*args)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/_pytest/runner.py", line 109, in pytest_runtest_protocol
INTERNALERROR>     runtestprotocol(item, nextitem=nextitem)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/_pytest/runner.py", line 126, in runtestprotocol
INTERNALERROR>     reports.append(call_and_report(item, "call", log))
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/_pytest/runner.py", line 221, in call_and_report
INTERNALERROR>     hook.pytest_exception_interact(node=item, call=call, report=report)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_hooks.py", line 265, in __call__
INTERNALERROR>     return self._hookexec(self.name, self.get_hookimpls(), kwargs, firstresult)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_manager.py", line 80, in _hookexec
INTERNALERROR>     return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_callers.py", line 60, in _multicall
INTERNALERROR>     return outcome.get_result()
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_result.py", line 60, in get_result
INTERNALERROR>     raise ex[1].with_traceback(ex[2])
INTERNALERROR>   File "<projectdir>/.venv/lib/python3.8/site-packages/pluggy/_callers.py", line 39, in _multicall
INTERNALERROR>     res = hook_impl.function(*args)
INTERNALERROR>   File "<homedir>/.vscode/extensions/ms-python.python-2023.14.0/pythonFiles/vscode_pytest/__init__.py", line 74, in pytest_exception_interact
INTERNALERROR>     if type(report) == pytest.CollectReport:
INTERNALERROR> AttributeError: module 'pytest' has no attribute 'CollectReport'
pytest session has finished, exit status:  ExitCode.INTERNAL_ERROR in discovery?  False

Steps to reproduce:

  1. Debug a failing pytest test using VS Code.

Diagnostic data

  • Python version (& distribution if applicable, e.g. Anaconda): 3.8.10
  • Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Poetry
  • Value of the python.languageServer setting: Default
Output for Python in the Output panel (ViewOutput, change the drop-down the upper-right of the Output panel to Python)

2023-08-08 16:05:11.883 [info] Experiment 'pythonPromptNewFormatterExt' is active
2023-08-08 16:05:11.883 [info] Experiment 'pythonPromptNewToolsExt' is active
2023-08-08 16:05:11.883 [info] Experiment 'pythonTestAdapter' is active
2023-08-08 16:05:11.883 [info] VS Code was launched from an activated environment: '.venv', selecting it as the interpreter for workspace.
2023-08-08 16:05:11.883 [info] Test server listening.
2023-08-08 16:05:11.883 [info] Python interpreter path: ./.venv/bin/python
2023-08-08 16:05:13.173 [info] > conda info --json
2023-08-08 16:05:13.449 [info] > /usr/share/anaconda/bin/conda info --json
2023-08-08 16:05:13.707 [info] > . ./.venv/bin/activate && echo 'e8b39361-0157-4923-80e1-22d70d46dee6' && python ~/.vscode/extensions/ms-python.python-2023.14.0/pythonFiles/printEnvVariables.py
2023-08-08 16:05:13.708 [info] shell: bash
2023-08-08 16:05:13.836 [info] Discover tests for workspace name: api-management-infrastructure - uri: /home/jon/APIM/repos/api-management-infrastructure
2023-08-08 16:05:13.836 [info] Running discovery for pytest using the new test adapter.
2023-08-08 16:05:13.858 [info] > ./.venv/bin/python -m pytest -p vscode_pytest --collect-only -v docker/proxygen
2023-08-08 16:05:13.858 [info] cwd: .
2023-08-08 16:05:13.967 [info] LINTING: Skipping linting from Python extension, since Flake8 extension is installed and enabled.
2023-08-08 16:05:13.967 [info] LINTING: Skipping linting from Python extension, since Pylint extension is installed and enabled.
2023-08-08 16:05:15.022 [info] Starting Pylance language server.
2023-08-08 16:05:16.654 [info] Test server connected to a client.
2023-08-08 16:05:16.728 [info] Using result resolver for discovery
2023-08-08 16:05:21.728 [info] Running PYTEST execution for the following test ids: docker/proxygen/proxygen/tests/integration/test_specs.py::TestApiSpecsEndpoints::test_spec_only_validation
2023-08-08 16:05:21.728 [info] Server listening on port 33945
2023-08-08 16:05:21.729 [info] Running DEBUG pytest with arguments: --rootdir /home/jon/APIM/repos/api-management-infrastructure -v --capture no

2023-08-08 16:05:21.731 [info] Using launch configuration in launch.json file.
2023-08-08 16:05:22.136 [info] DAP Server launched with command: /home/jon/APIM/repos/api-management-infrastructure/.venv/bin/python /home/jon/.vscode/extensions/ms-python.python-2023.14.0/pythonFiles/lib/python/debugpy/adapter
2023-08-08 16:05:22.486 [info] Send text to terminal: source /home/jon/APIM/repos/api-management-infrastructure/.venv/bin/activate
2023-08-08 16:05:23.488 [info] Send text to terminal:  /usr/bin/env /home/jon/APIM/repos/api-management-infrastructure/.venv/bin/python /home/jon/.vscode/extensions/ms-python.python-2023.14.0/pythonFiles/lib/python/debugpy/adapter/../../debugpy/launcher 56395 -- /home/jon/.vscode/extensions/ms-python.python-2023.14.0/pythonFiles/vscode_pytest/run_pytest_script.py --rootdir /home/jon/APIM/repos/api-management-infrastructure -v --capture no 
2023-08-08 16:05:30.391 [info] Test server connected to a client.
2023-08-08 16:05:30.404 [info] Test server connected to a client.
2023-08-08 16:05:31.284 [info] Client disconnected

User Settings


languageServer: "Pylance"

linting
• flake8Enabled: true

formatting
• provider: "none"

testing
• pytestArgs: "<placeholder>"
• pytestEnabled: true
• pytestPath: "<placeholder>"

Extension version: 2023.14.0
VS Code version: Code 1.81.0 (6445d93c81ebe42c4cbd7a60712e0b17d9463e97, 2023-08-02T12:36:11.334Z)
OS version: Linux x64 6.4.7-200.fc38.x86_64
Modes:

System Info
Item Value
CPUs Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 3497)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
video_decode: enabled
video_encode: disabled_software
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: disabled_off
Load (avg) 1, 1, 1
Memory (System) 62.43GB (48.92GB free)
Process Argv . --crash-reporter-id ed7cda52-d9be-4e64-8def-511b71321e86
Screen Reader no
VM 0%
DESKTOP_SESSION gnome-xorg
XDG_CURRENT_DESKTOP GNOME
XDG_SESSION_DESKTOP gnome-xorg
XDG_SESSION_TYPE x11
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263cf:30335440
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263:30792226
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30803845
2e4cg342:30602488
f6dab269:30613381
showlangstatbar:30737416
vsctsb:30748421
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
gsofa:30804715
pythonnosmt12:30797651
pythonidxptcf:30805731
pythonnoceb:30805159
e537b577:30795824
dsvsc013:30795093
dsvsc014:30804076

@github-actions github-actions bot added the triage-needed Needs assignment to the proper sub-team label Aug 8, 2023
@johnmarston-nhs johnmarston-nhs changed the title Pytest Report failes due to AttributeError Pytest Report fails due to AttributeError Aug 8, 2023
@karthiknadig
Copy link
Member

@johnmarston-nhs What is the version of pytest that you have?

@github-actions github-actions bot added the info-needed Issue requires more information from poster label Aug 8, 2023
@karthiknadig karthiknadig added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Aug 8, 2023
@johnmarston-nhs
Copy link
Author

@karthiknadig 6.2.5.

Just to add to this, it works fine if I pin the version of vscode-python to 2023.12.0.

@eleanorjboyd
Copy link
Member

Thank you for your issue report. If you add this to your settings does this fix it? user settings: "python.experiments.optOutFrom": ["pythonTestAdapter"],. If that does fix it then it is likely on the rewrite which will give me more information I can use to fix this. Thanks

@github-actions github-actions bot added the info-needed Issue requires more information from poster label Aug 9, 2023
@johnmarston-nhs
Copy link
Author

johnmarston-nhs commented Aug 9, 2023

Hi @eleanorjboyd , yes, opting out from pythonTestAdapter fixes the issue, thank you for your suggestion. Is leaving this disabled gonna bite me in the butt later down the line? 😄

As an addition, I was looking into the exception - it seems to be down to the CollectReport class not being 'exported' in versions of pytest below v7. It got added in this commit: pytest-dev/pytest@b0aa870

Maybe you can work around this with a str() around the call to type() or something, as I think pytest 6 is still quite widely used.

@github-actions github-actions bot removed the info-needed Issue requires more information from poster label Aug 9, 2023
@eleanorjboyd
Copy link
Member

Thank you for that investigation! I will look into that as a workaround.

Also no, the setting shouldn't be a problem later as once we are fully confident in the re-write and move it to 100% of stable we will get rid of the experiment and that setting will be ignored.

@eleanorjboyd eleanorjboyd added needs PR Ready to be worked on and removed triage-needed Needs assignment to the proper sub-team labels Aug 11, 2023
@eleanorjboyd eleanorjboyd added this to the August 2023 milestone Aug 21, 2023
eleanorjboyd added a commit that referenced this issue Aug 21, 2023
as per
https://docs.pytest.org/en/7.1.x/reference/reference.html#collectreport,
`CollectReport` is experimental and therefore it should not be in our
extension. Fixes #21784
@github-actions github-actions bot removed the needs PR Ready to be worked on label Aug 21, 2023
@eleanorjboyd eleanorjboyd added the verification-needed Verification of issue is requested label Aug 28, 2023
@andreamah andreamah added the verification-steps-needed Steps to verify are needed for verification label Aug 29, 2023
@andreamah
Copy link

Are there verification steps? Or @johnmarston-nhs, do you still see this in the latest Python prerelease?

@eleanorjboyd
Copy link
Member

  1. set your pytest version to 6.2.5
  2. create a test in a file like test_file.py
def test_a:
     assert True
  1. run pytest discovery (testing tab, configure testing with pytest)
  2. run this pytest test
  3. confirm you do not see any errors like above in your "python testing log" section of the output panel

@eleanorjboyd eleanorjboyd removed the verification-steps-needed Steps to verify are needed for verification label Aug 29, 2023
@sandy081 sandy081 added the verified Verification succeeded label Aug 30, 2023
@sandy081
Copy link
Member

To verify I have to update the above sample to following

def test_a():
     assert True

@collinwr
Copy link

collinwr commented Sep 1, 2023

I am running pytest 6.2.5, and I am seeing the error even with the "python.experiments.optOutFrom": ["pythonTestAdapter"], setting configured in my user settings.

@eleanorjboyd
Copy link
Member

to confirm @collinwr you are seeing INTERNALERROR> AttributeError: module 'pytest' has no attribute 'CollectReport' right? Do you see this even when you opt into the experiment? It will currently only work on insiders as this is unreleased in stable.

anthonykim1 pushed a commit to anthonykim1/vscode-python that referenced this issue Sep 12, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 6, 2023
wesm pushed a commit to posit-dev/positron that referenced this issue Mar 28, 2024
…se 2023.16.0

Merge pull request #213 from posit-dev/merge/2023.16.0

Merge python extension release 2023.16.0
--------------------
Commit message for posit-dev/positron-python@cca4be0:

Fix pyright issues

--------------------
Commit message for posit-dev/positron-python@c3bd360:

Merge branch 'main' into merge/2023.16.0

--------------------
Commit message for posit-dev/positron-python@bebadea:

EnvironmentVariableScope now provided by vscode 1.82

--------------------
Commit message for posit-dev/positron-python@3e9d86b:

Add back installation capability for ipykernel

--------------------
Commit message for posit-dev/positron-python@6a7d49a:

Merge commit '8c612511b99cb5ebc78153684de8fe595bd154dc'

--------------------
Commit message for microsoft/vscode-python@8c61251:

Update version for release candidate (microsoft/vscode-python#21919)


--------------------
Commit message for microsoft/vscode-python@d9b9c88:

Always prepend to PATH instead of replacing it (microsoft/vscode-python#21906)

For microsoft/vscode-python#20822 microsoft/vscode-python#11039

Replacing as-is has its problems, for eg. pyenv asks their users to
manipulate `PATH` in their init script:
https://github.com/pyenv/pyenv#set-up-your-shell-environment-for-pyenv,
which we could end up replacing.


![image](https://github.com/microsoft/vscode-python/assets/13199757/cc904f76-8d42-47e1-a6c8-6cfff6543db8)

Particularly for pyenv, it mean users not being able to find pyenv:


![image](https://github.com/microsoft/vscode-python/assets/13199757/26100328-c227-435b-a4f2-ec168099f4c1)

Prepending solves it for cases where initial PATH value is suffix of the
final value:


![image](https://github.com/microsoft/vscode-python/assets/13199757/a95e4ffd-68dc-4e73-905e-504b3051324f)

But, in other cases, this means that we end up with the whole `PATH`
thrice. This is because it prepends it twice:
- Once in shell integration script
- Once when creating a process

So the final value could be:
```
PATH=<activated_full_path><activated_full_path><original_path>
```
where `<activated_full_path>` refers to value of `PATH` environment
variable post activation. eg.


![image](https://github.com/microsoft/vscode-python/assets/13199757/7e771f62-eb53-49be-b261-d259096008f3)
--------------------
Commit message for microsoft/vscode-python@7a9294c:

Apply feedback for terminal activation prompt (microsoft/vscode-python#21905)

For microsoft/vscode-python#21793


![image](https://github.com/microsoft/vscode-python/assets/13199757/b3ab6002-0a07-4b3b-8101-a84865ea12e4)
--------------------
Commit message for microsoft/vscode-python@44f5bf7:

Set PS1 for conda environments in non-Windows when in `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21902)

For microsoft/vscode-python#20822 


![image](https://github.com/microsoft/vscode-python/assets/13199757/8c9d4c87-54f2-4661-b6c6-c3b49ee3ff7a)
--------------------
Commit message for microsoft/vscode-python@7d25ceb:

Remove finalized api proposals from package.json (microsoft/vscode-python#21900)

Part of microsoft/vscode#191605
--------------------
Commit message for microsoft/vscode-python@31aa246:

Also show env name for prefixed conda envs in terminal prompt (microsoft/vscode-python#21899)


--------------------
Commit message for microsoft/vscode-python@941fcfa:

Fixes from TPIs (microsoft/vscode-python#21896)

Closes microsoft/vscode-python#21884
Closes microsoft/vscode-python#21889
--------------------
Commit message for microsoft/vscode-python@f255e02:

Call out that env name may not show in terminal activation notification (microsoft/vscode-python#21897)

Closes microsoft/vscode-python#21887
--------------------
Commit message for microsoft/vscode-python@782d5b1:

Also show interpreter in status bar when a Python related output channel is opened (microsoft/vscode-python#21894)

Closes microsoft/vscode-python#21890
--------------------
Commit message for microsoft/vscode-python@1204011:

Activate environment when not using integrated terminal for debugging (microsoft/vscode-python#21880)

For microsoft/vscode-python#4300
--------------------
Commit message for microsoft/vscode-python@98428cd:

Apply custom env variables to terminal when in `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21879)

For microsoft/vscode-python#944 microsoft/vscode-python#20822 

We only apply those env vars to terminal which are not in process env
variables, hence remove custom env vars from process variables.
--------------------
Commit message for microsoft/vscode-python@3fa5d4b:

Support for Create Env command to re-create env for venv (microsoft/vscode-python#21829)

Closes microsoft/vscode-python#21827
--------------------
Commit message for microsoft/vscode-python@30e26c2:

Update proposed API for env collection (microsoft/vscode-python#21819)

For microsoft/vscode-python#20822

Blocked on
microsoft/vscode#171173 (comment)
--------------------
Commit message for microsoft/vscode-python@15bb974:

Do not filter using scheme when filtering environments (microsoft/vscode-python#21862)

For microsoft/vscode-python#21825

On codespaces, it was leading to workspace environments not being
displayed, which could mess up auto-selection.
--------------------
Commit message for microsoft/vscode-python@cfbf1f3:

remove usage of pytest CollectReport in rewrite (microsoft/vscode-python#21859)

as per
https://docs.pytest.org/en/7.1.x/reference/reference.html#collectreport,
`CollectReport` is experimental and therefore it should not be in our
extension. Fixes microsoft/vscode-python#21784
--------------------
Commit message for microsoft/vscode-python@0749b20:

Show `Python: Clear Workspace interpreter` command regardless of whether a Python file is opened (microsoft/vscode-python#21858)

Closes microsoft/vscode-python#21850
--------------------
Commit message for microsoft/vscode-python@021b362:

Update VS Code engine (microsoft/vscode-python#21847)

For microsoft/vscode-python#21831
--------------------
Commit message for microsoft/vscode-python@8407e9d:

Wrap env collection workspace proposed APIs in `try...catch` block (microsoft/vscode-python#21846)

Closes microsoft/vscode-python#21831
--------------------
Commit message for microsoft/vscode-python@c979455:

Set workspaceFolder in debug config before substituting command variables (microsoft/vscode-python#21835)

For microsoft/vscode-python#18482
--------------------
Commit message for microsoft/vscode-python@96ba735:

fix data to string from buffer for output channel (microsoft/vscode-python#21821)

fix microsoft/vscode-python#21820
--------------------
Commit message for microsoft/vscode-python@5140a8d:

Apply API recommendations for Create Env API (microsoft/vscode-python#21804)

Closes microsoft/vscode-python#21090
--------------------
Commit message for microsoft/vscode-python@0248fa8:

fixing failing tests on CI (microsoft/vscode-python#21814)

fixing microsoft/vscode-python#21813
--------------------
Commit message for microsoft/vscode-python@9c740b9:

Show notification reaffirming Python extension still handles activation when in `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21802)

Closes microsoft/vscode-python#21793

Only show notification when terminal prompt does not already indicate
that env is activated.
--------------------
Commit message for microsoft/vscode-python@b447bf1:

Feature branch testing overflow bug fix (microsoft/vscode-python#21812)

This merges in two PRs that were reverted because of a bug introduced
that caused subprocess overflow.
reverted PRs: microsoft/vscode-python#21667,
microsoft/vscode-python#21682

This now implements these two PRs allowing for absolute testIds and an
execObservable for the subprocess. This PR also adds a bug fix and
functional tests to ensure this doesn't happen again.

Since this PR is large, all items in it have already been reviewed as
they were merged into the feature branch.
--------------------
Commit message for microsoft/vscode-python@bd749aa:

Fix `service.test.ts` to stop disposing of all services (microsoft/vscode-python#21811)

file `service.test.ts` was calling to dispose of all items related to
the service container for clean up. This led to services in later tests
failing since they were close already. Fixes here allow for new tests in
the test adapter to be written.

fix helps microsoft/vscode-python#21803
--------------------
Commit message for microsoft/vscode-python@385bb37:

Add `language_server.jinja_usage` to `pylance.ts` (microsoft/vscode-python#21809)


--------------------
Commit message for microsoft/vscode-python@71d6dab:

Add one more property to load event (microsoft/vscode-python#21800)

This PR adds app name to the editor_load telemetry event
--------------------
Commit message for microsoft/vscode-python@ab8d3b2:

Update VS Code engine (microsoft/vscode-python#21799)

For microsoft/vscode-python#11039
--------------------
Commit message for microsoft/vscode-python@835eab5:

Add setting to control severity of missing package diagnostic. (microsoft/vscode-python#21794)

Closes microsoft/vscode-python#21792
--------------------
Commit message for microsoft/vscode-python@fbbf987:

Use updated API to fetch scoped env collection (microsoft/vscode-python#21788)

For microsoft/vscode#171173
microsoft/vscode-python#20822

To be merged tomorrow when latest insiders is released. Blocked on
microsoft/vscode#189979.
--------------------
Commit message for microsoft/vscode-python@0a2c285:

Bump brettcannon/check-for-changed-files from 1.1.1 to 1.2.0 (microsoft/vscode-python#21772)

Bumps
[brettcannon/check-for-changed-files](https://github.com/brettcannon/check-for-changed-files)
from 1.1.1 to 1.2.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/brettcannon/check-for-changed-files/releases">brettcannon/check-for-changed-files's
releases</a>.</em></p>
<blockquote>
<h2>v1.2.0</h2>
<h2>What's Changed</h2>
<ul>
<li>Add support for github tokens for private repos. by <a
href="https://github.com/Marenz"><code>@​Marenz</code></a> in <a
href="https://redirect.github.com/brettcannon/check-for-changed-files/pull/83">brettcannon/check-for-changed-files#83</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a href="https://github.com/Marenz"><code>@​Marenz</code></a> made
their first contribution in <a
href="https://redirect.github.com/brettcannon/check-for-changed-files/pull/83">brettcannon/check-for-changed-files#83</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/brettcannon/check-for-changed-files/compare/v1...v1.2.0">https://github.com/brettcannon/check-for-changed-files/compare/v1...v1.2.0</a></p>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/brettcannon/check-for-changed-files/commit/4170644959a21843b31f1181f2a1761d65ef4791"><code>4170644</code></a>
Add support for github tokens for private repos. (<a
href="https://redirect.github.com/brettcannon/check-for-changed-files/issues/83">#83</a>)</li>
<li><a
href="https://github.com/brettcannon/check-for-changed-files/commit/294a99714e0d350b5083472a293d41bc91804e68"><code>294a997</code></a>
Bump word-wrap from 1.2.3 to 1.2.4 (<a
href="https://redirect.github.com/brettcannon/check-for-changed-files/issues/82">#82</a>)</li>
<li>See full diff in <a
href="https://github.com/brettcannon/check-for-changed-files/compare/v1.1.1...v1.2.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=brettcannon/check-for-changed-files&package-manager=github_actions&previous-version=1.1.1&new-version=1.2.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 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] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
--------------------
Commit message for microsoft/vscode-python@9ac14b8:

Update README.md for npm package (microsoft/vscode-python#21766)

Fix indent for
https://www.npmjs.com/package/@vscode/python-extension?activeTab=readme
--------------------
Commit message for microsoft/vscode-python@3fed49f:

Add Ruff to CI (microsoft/vscode-python#21739)

Add Ruff to (lint > action.yml) CI
#21738
--------------------
Commit message for microsoft/vscode-python@c490339:

Update version of npm package (microsoft/vscode-python#21765)


--------------------
Commit message for microsoft/vscode-python@8e0e59b:

Remove optionalDependencies from API npm package and document to install vscode types separately (microsoft/vscode-python#21764)

Closes It still leads to conflicts due to double installation of vscode
types when testing through the cases, removing vscode types as
dependencies altogether and documenting to install it separately
instead.
--------------------
Commit message for microsoft/vscode-python@cabdf39:

Use `optionalDependencies` instead of `peerDependencies` for `@vscode/python-extension` npm package (microsoft/vscode-python#21763)

Closes microsoft/vscode-python#21720
--------------------
Commit message for microsoft/vscode-python@dd20561:

revert due to buffer overflow on subprocess (microsoft/vscode-python#21762)

revert microsoft/vscode-python#21667 because it
causes buffer overflow in the python testing subprocess when larger
repos are used. Specifically seen on pytest discovery with >200 tests.
Revert to align with the stable release and put in a fix next week.
--------------------
Commit message for microsoft/vscode-python@40ff6e9:

Remove private Jupyter APIs from public API types (microsoft/vscode-python#21761)

For microsoft/vscode-jupyter#13986
--------------------
Commit message for microsoft/vscode-python@23353bb:

Improvements to `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21751)


--------------------
Commit message for microsoft/vscode-python@40bb62a:

fix spelling for get-pip.py (microsoft/vscode-python#21752)

fix spelling from get_pip to get-pip as advised.
--------------------
Commit message for microsoft/vscode-python@f454515:

Update release plan to document what to do with `main` during endgame week (microsoft/vscode-python#21743)


--------------------
Commit message for microsoft/vscode-python@dff25d4:

revert absolute test-ids (microsoft/vscode-python#21742)

seeing a substantial error where test discovery is broken. Reverting
this commit seems to be the temporary fix until I can diagnose the real
problem.

commit it is reverting:
microsoft/vscode-python#21682
--------------------
Commit message for microsoft/vscode-python@ca4dfd4:

update tests only on save with more files excluded (microsoft/vscode-python#21741)

fixes microsoft/vscode-python#21014 and
microsoft/vscode-python#21061
--------------------
Commit message for microsoft/vscode-python@84bbff9:

add cwd for debugging (microsoft/vscode-python#21668)

fixes microsoft/vscode-python#21648 (comment)
--------------------
Commit message for microsoft/vscode-python@ef16727:

Clean up smoke test requirement (microsoft/vscode-python#21729)

Cleaning up smoke test dependency: See if all Github action test pass
with removing the smoke test requirement file content.

Checked one by one, and came to see removing all doesn't seem to have
impact on the outcome of running smoke test.(Seems to have no difference
in smoke test result outcome when ran with "Run and Debug" in VS Code
with smoke-test option selected).

Also got rid of below, after checking smoke test correctly passing after
removal of smoke-test-requirement.txt content:
![Screenshot 2023-08-01 at 2 57 45
PM](https://github.com/microsoft/vscode-python/assets/62267334/45d404de-74dd-45a5-885b-71a25ef16ad7)

Resolve: microsoft/vscode-python#21496
--------------------
Commit message for microsoft/vscode-python@358635d:

Remove old and unused API for Jupyter Ext (microsoft/vscode-python#21731)

We have not used any of this API for a while now, hence its safe to
remove these.

Will be removing more soon.
--------------------
Commit message for microsoft/vscode-python@8f3d60b:

unittest discovery errors not displaying in test explorer (microsoft/vscode-python#21726)

saw an issue where if discovery failed there was no notice in the test
explorer for unittest. It was due to a different value for the new blank
value for the payload tests. fixes
microsoft/vscode-python#21725 and
microsoft/vscode-python#21688
--------------------
Commit message for microsoft/vscode-python@a6a8cb1:

Update main to next pre-release (microsoft/vscode-python#21728)


--------------------
Commit message for microsoft/vscode-python@4ab510d:

Update version for release candidate (microsoft/vscode-python#21727)


--------------------
Commit message for microsoft/vscode-python@3e7118f:

Update packages for Jedi and core python (microsoft/vscode-python#21710)


--------------------
Commit message for microsoft/vscode-python@237f82b:

Fix UUID and disposing to resolve race condition (microsoft/vscode-python#21667)

fixes microsoft/vscode-python#21599 and
microsoft/vscode-python#21507
--------------------
Commit message for microsoft/vscode-python@d9e368f:

add area-repl to issue label (microsoft/vscode-python#21718)

added area-repl as one of the issue label.
--------------------
Commit message for microsoft/vscode-python@ceecdb0:

Removing Jupyter Notebooks mentions from package.json (microsoft/vscode-python#21708)


--------------------
Commit message for microsoft/vscode-python@11a9f1d:

Remove unwanted Jupyter API (microsoft/vscode-python#21702)

Fixes microsoft/vscode-jupyter#13986
--------------------
Commit message for microsoft/vscode-python@efcc3d7:

Make test_ids relative to workspace path not root dir (microsoft/vscode-python#21682)

makes sure all testIds that are returned to the extension are relative
to the workspace (which will be the invocation directory) instead of to
the root. This will stop testIds for not being recognized when using a
config file or another parameter that changes the root directory during
pytest.

fixes microsoft/vscode-python#21640 and
microsoft/vscode-python#21637
--------------------
Commit message for microsoft/vscode-python@06d62aa:

Update homepage for Python API package (microsoft/vscode-python#21703)

For microsoft/vscode-python#21631
--------------------
Commit message for microsoft/vscode-python@83107cc:

Move "vscode" out of required dependencies for npm package (microsoft/vscode-python#21701)

Closes microsoft/vscode-python#21684
--------------------
Commit message for microsoft/vscode-python@d673004:

Convert JS-style typings to native TS in `@vscode/python-extension` (microsoft/vscode-python#21692)

Closes microsoft/vscode-python#21690
--------------------
Commit message for microsoft/vscode-python@8b9bca1:

Do not show "Select at workspace level" option if only one workspace folder is opened (microsoft/vscode-python#21689)

Closes microsoft/vscode-python#21220
--------------------
Commit message for microsoft/vscode-python@f536b74:

Edit issue-labels.yml, triage-info-needed.yml (microsoft/vscode-python#21685)

Add Anthony to issue-labels.yml and triage-info-needed.yml
--------------------
Commit message for microsoft/vscode-python@a42cb33:

Add new telemetry property to GPDR (microsoft/vscode-python#21683)

This property was added for tracking diagnostics we emit in Pylance.
--------------------
Commit message for microsoft/vscode-python@6af959d:

Dev Container Using MCR (microsoft/vscode-python#21675)

Dev container rewrite using MCR. 
Pyenv for installing and managing python versions. Fish also installed
as optional (able to view as shell option in codespaces). Also fixing
conda error. Takes care of: microsoft/vscode-python#21591 rewrite from: microsoft/vscode-python#21435 to adhere to
company policy.
--------------------
Commit message for microsoft/vscode-python@73a0e9d:

handle skip unittest at file without error (microsoft/vscode-python#21678)

fixes microsoft/vscode-python#21653
--------------------
Commit message for microsoft/vscode-python@9bcb82d:

Ensure `Run Python in dedicated terminal` uses `python.executeInFirDir` setting (microsoft/vscode-python#21681)


--------------------
Commit message for microsoft/vscode-python@713007f:

correct discovery on unittest skip at file level (microsoft/vscode-python#21665)

given a file called skip_test_file_node.py that has `raise
SkipTest(".....")` this should appear in the sidebar with no children.
The bug is that currently it shows a "unittest" node that gives "loader"
and other incorrect nodes below it.
--------------------
Commit message for microsoft/vscode-python@be334bd:

Do not resolve symbolic links in posix locator if they exceed the count limit (microsoft/vscode-python#21658)

Closes microsoft/vscode-python#21310

Fixes interpreter discovery running forever for non-Windows OS
--------------------
Commit message for microsoft/vscode-python@c256678:

Prevent posix paths locator from crashing (microsoft/vscode-python#21657)

For microsoft/vscode-python#21310
--------------------
Commit message for microsoft/vscode-python@81ae205:

Bring back feature to Run Python file in dedicated terminal (microsoft/vscode-python#21656)

Closes microsoft/vscode-python#21282
Closes microsoft/vscode-python#21420
Closes microsoft/vscode-python#21215
Reverts microsoft/vscode-python#21418
--------------------
Commit message for microsoft/vscode-python@c144200:

Modify .eslintrc to turn off any errors for declaration files (microsoft/vscode-python#21652)

For microsoft/vscode-python#21631
--------------------
Commit message for microsoft/vscode-python@f7125da:

Use correct `tsconfig.json` when generating npm package (microsoft/vscode-python#21651)

For microsoft/vscode-python#21631 

- Unset `removeComment` as that leads to declarations without docstrings
- Set to generate declarations
- Use updated typescript which results in cleaner declaration files
--------------------
Commit message for microsoft/vscode-python@2e8dc67:

Add extra logging regarding interpreter discovery (microsoft/vscode-python#21639)

For microsoft/vscode-python#21310
--------------------
Commit message for microsoft/vscode-python@fc1c391:

Compare global storage data using only `key` (microsoft/vscode-python#21636)

Closes microsoft/vscode-python#21635 by
applying the same fix as done in
microsoft/vscode-python#17627.

Lead-authored-by: Kartik Raj <[email protected]>
Co-authored-by: Daniel Imms <[email protected]>
Co-authored-by: Luciana Abud <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Courtney Webster <[email protected]>
Co-authored-by: Brett Cannon <[email protected]>
Co-authored-by: Erik De Bonte <[email protected]>
Co-authored-by: Rich Chiodo <[email protected]>
Co-authored-by: Don Jayamanne <[email protected]>
Co-authored-by: Anthony Kim <[email protected]>
Co-authored-by: Pete Farland <[email protected]>
Co-authored-by: Karthik Nadig <[email protected]>
Co-authored-by: Eleanor Boyd <[email protected]>
Signed-off-by: GitHub <[email protected]>
wesm pushed a commit to posit-dev/positron that referenced this issue Mar 28, 2024
…se 2023.16.0

Merge pull request #213 from posit-dev/merge/2023.16.0

Merge python extension release 2023.16.0
--------------------
Commit message for posit-dev/positron-python@cca4be0:

Fix pyright issues

--------------------
Commit message for posit-dev/positron-python@c3bd360:

Merge branch 'main' into merge/2023.16.0

--------------------
Commit message for posit-dev/positron-python@bebadea:

EnvironmentVariableScope now provided by vscode 1.82

--------------------
Commit message for posit-dev/positron-python@3e9d86b:

Add back installation capability for ipykernel

--------------------
Commit message for posit-dev/positron-python@6a7d49a:

Merge commit '8c612511b99cb5ebc78153684de8fe595bd154dc'

--------------------
Commit message for microsoft/vscode-python@8c61251:

Update version for release candidate (microsoft/vscode-python#21919)


--------------------
Commit message for microsoft/vscode-python@d9b9c88:

Always prepend to PATH instead of replacing it (microsoft/vscode-python#21906)

For microsoft/vscode-python#20822 microsoft/vscode-python#11039

Replacing as-is has its problems, for eg. pyenv asks their users to
manipulate `PATH` in their init script:
https://github.com/pyenv/pyenv#set-up-your-shell-environment-for-pyenv,
which we could end up replacing.


![image](https://github.com/microsoft/vscode-python/assets/13199757/cc904f76-8d42-47e1-a6c8-6cfff6543db8)

Particularly for pyenv, it mean users not being able to find pyenv:


![image](https://github.com/microsoft/vscode-python/assets/13199757/26100328-c227-435b-a4f2-ec168099f4c1)

Prepending solves it for cases where initial PATH value is suffix of the
final value:


![image](https://github.com/microsoft/vscode-python/assets/13199757/a95e4ffd-68dc-4e73-905e-504b3051324f)

But, in other cases, this means that we end up with the whole `PATH`
thrice. This is because it prepends it twice:
- Once in shell integration script
- Once when creating a process

So the final value could be:
```
PATH=<activated_full_path><activated_full_path><original_path>
```
where `<activated_full_path>` refers to value of `PATH` environment
variable post activation. eg.


![image](https://github.com/microsoft/vscode-python/assets/13199757/7e771f62-eb53-49be-b261-d259096008f3)
--------------------
Commit message for microsoft/vscode-python@7a9294c:

Apply feedback for terminal activation prompt (microsoft/vscode-python#21905)

For microsoft/vscode-python#21793


![image](https://github.com/microsoft/vscode-python/assets/13199757/b3ab6002-0a07-4b3b-8101-a84865ea12e4)
--------------------
Commit message for microsoft/vscode-python@44f5bf7:

Set PS1 for conda environments in non-Windows when in `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21902)

For microsoft/vscode-python#20822 


![image](https://github.com/microsoft/vscode-python/assets/13199757/8c9d4c87-54f2-4661-b6c6-c3b49ee3ff7a)
--------------------
Commit message for microsoft/vscode-python@7d25ceb:

Remove finalized api proposals from package.json (microsoft/vscode-python#21900)

Part of microsoft/vscode#191605
--------------------
Commit message for microsoft/vscode-python@31aa246:

Also show env name for prefixed conda envs in terminal prompt (microsoft/vscode-python#21899)


--------------------
Commit message for microsoft/vscode-python@941fcfa:

Fixes from TPIs (microsoft/vscode-python#21896)

Closes microsoft/vscode-python#21884
Closes microsoft/vscode-python#21889
--------------------
Commit message for microsoft/vscode-python@f255e02:

Call out that env name may not show in terminal activation notification (microsoft/vscode-python#21897)

Closes microsoft/vscode-python#21887
--------------------
Commit message for microsoft/vscode-python@782d5b1:

Also show interpreter in status bar when a Python related output channel is opened (microsoft/vscode-python#21894)

Closes microsoft/vscode-python#21890
--------------------
Commit message for microsoft/vscode-python@1204011:

Activate environment when not using integrated terminal for debugging (microsoft/vscode-python#21880)

For microsoft/vscode-python#4300
--------------------
Commit message for microsoft/vscode-python@98428cd:

Apply custom env variables to terminal when in `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21879)

For microsoft/vscode-python#944 microsoft/vscode-python#20822 

We only apply those env vars to terminal which are not in process env
variables, hence remove custom env vars from process variables.
--------------------
Commit message for microsoft/vscode-python@3fa5d4b:

Support for Create Env command to re-create env for venv (microsoft/vscode-python#21829)

Closes microsoft/vscode-python#21827
--------------------
Commit message for microsoft/vscode-python@30e26c2:

Update proposed API for env collection (microsoft/vscode-python#21819)

For microsoft/vscode-python#20822

Blocked on
microsoft/vscode#171173 (comment)
--------------------
Commit message for microsoft/vscode-python@15bb974:

Do not filter using scheme when filtering environments (microsoft/vscode-python#21862)

For microsoft/vscode-python#21825

On codespaces, it was leading to workspace environments not being
displayed, which could mess up auto-selection.
--------------------
Commit message for microsoft/vscode-python@cfbf1f3:

remove usage of pytest CollectReport in rewrite (microsoft/vscode-python#21859)

as per
https://docs.pytest.org/en/7.1.x/reference/reference.html#collectreport,
`CollectReport` is experimental and therefore it should not be in our
extension. Fixes microsoft/vscode-python#21784
--------------------
Commit message for microsoft/vscode-python@0749b20:

Show `Python: Clear Workspace interpreter` command regardless of whether a Python file is opened (microsoft/vscode-python#21858)

Closes microsoft/vscode-python#21850
--------------------
Commit message for microsoft/vscode-python@021b362:

Update VS Code engine (microsoft/vscode-python#21847)

For microsoft/vscode-python#21831
--------------------
Commit message for microsoft/vscode-python@8407e9d:

Wrap env collection workspace proposed APIs in `try...catch` block (microsoft/vscode-python#21846)

Closes microsoft/vscode-python#21831
--------------------
Commit message for microsoft/vscode-python@c979455:

Set workspaceFolder in debug config before substituting command variables (microsoft/vscode-python#21835)

For microsoft/vscode-python#18482
--------------------
Commit message for microsoft/vscode-python@96ba735:

fix data to string from buffer for output channel (microsoft/vscode-python#21821)

fix microsoft/vscode-python#21820
--------------------
Commit message for microsoft/vscode-python@5140a8d:

Apply API recommendations for Create Env API (microsoft/vscode-python#21804)

Closes microsoft/vscode-python#21090
--------------------
Commit message for microsoft/vscode-python@0248fa8:

fixing failing tests on CI (microsoft/vscode-python#21814)

fixing microsoft/vscode-python#21813
--------------------
Commit message for microsoft/vscode-python@9c740b9:

Show notification reaffirming Python extension still handles activation when in `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21802)

Closes microsoft/vscode-python#21793

Only show notification when terminal prompt does not already indicate
that env is activated.
--------------------
Commit message for microsoft/vscode-python@b447bf1:

Feature branch testing overflow bug fix (microsoft/vscode-python#21812)

This merges in two PRs that were reverted because of a bug introduced
that caused subprocess overflow.
reverted PRs: microsoft/vscode-python#21667,
microsoft/vscode-python#21682

This now implements these two PRs allowing for absolute testIds and an
execObservable for the subprocess. This PR also adds a bug fix and
functional tests to ensure this doesn't happen again.

Since this PR is large, all items in it have already been reviewed as
they were merged into the feature branch.
--------------------
Commit message for microsoft/vscode-python@bd749aa:

Fix `service.test.ts` to stop disposing of all services (microsoft/vscode-python#21811)

file `service.test.ts` was calling to dispose of all items related to
the service container for clean up. This led to services in later tests
failing since they were close already. Fixes here allow for new tests in
the test adapter to be written.

fix helps microsoft/vscode-python#21803
--------------------
Commit message for microsoft/vscode-python@385bb37:

Add `language_server.jinja_usage` to `pylance.ts` (microsoft/vscode-python#21809)


--------------------
Commit message for microsoft/vscode-python@71d6dab:

Add one more property to load event (microsoft/vscode-python#21800)

This PR adds app name to the editor_load telemetry event
--------------------
Commit message for microsoft/vscode-python@ab8d3b2:

Update VS Code engine (microsoft/vscode-python#21799)

For microsoft/vscode-python#11039
--------------------
Commit message for microsoft/vscode-python@835eab5:

Add setting to control severity of missing package diagnostic. (microsoft/vscode-python#21794)

Closes microsoft/vscode-python#21792
--------------------
Commit message for microsoft/vscode-python@fbbf987:

Use updated API to fetch scoped env collection (microsoft/vscode-python#21788)

For microsoft/vscode#171173
microsoft/vscode-python#20822

To be merged tomorrow when latest insiders is released. Blocked on
microsoft/vscode#189979.
--------------------
Commit message for microsoft/vscode-python@0a2c285:

Bump brettcannon/check-for-changed-files from 1.1.1 to 1.2.0 (microsoft/vscode-python#21772)

Bumps
[brettcannon/check-for-changed-files](https://github.com/brettcannon/check-for-changed-files)
from 1.1.1 to 1.2.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/brettcannon/check-for-changed-files/releases">brettcannon/check-for-changed-files's
releases</a>.</em></p>
<blockquote>
<h2>v1.2.0</h2>
<h2>What's Changed</h2>
<ul>
<li>Add support for github tokens for private repos. by <a
href="https://github.com/Marenz"><code>@​Marenz</code></a> in <a
href="https://redirect.github.com/brettcannon/check-for-changed-files/pull/83">brettcannon/check-for-changed-files#83</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a href="https://github.com/Marenz"><code>@​Marenz</code></a> made
their first contribution in <a
href="https://redirect.github.com/brettcannon/check-for-changed-files/pull/83">brettcannon/check-for-changed-files#83</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/brettcannon/check-for-changed-files/compare/v1...v1.2.0">https://github.com/brettcannon/check-for-changed-files/compare/v1...v1.2.0</a></p>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/brettcannon/check-for-changed-files/commit/4170644959a21843b31f1181f2a1761d65ef4791"><code>4170644</code></a>
Add support for github tokens for private repos. (<a
href="https://redirect.github.com/brettcannon/check-for-changed-files/issues/83">#83</a>)</li>
<li><a
href="https://github.com/brettcannon/check-for-changed-files/commit/294a99714e0d350b5083472a293d41bc91804e68"><code>294a997</code></a>
Bump word-wrap from 1.2.3 to 1.2.4 (<a
href="https://redirect.github.com/brettcannon/check-for-changed-files/issues/82">#82</a>)</li>
<li>See full diff in <a
href="https://github.com/brettcannon/check-for-changed-files/compare/v1.1.1...v1.2.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=brettcannon/check-for-changed-files&package-manager=github_actions&previous-version=1.1.1&new-version=1.2.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 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] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
--------------------
Commit message for microsoft/vscode-python@9ac14b8:

Update README.md for npm package (microsoft/vscode-python#21766)

Fix indent for
https://www.npmjs.com/package/@vscode/python-extension?activeTab=readme
--------------------
Commit message for microsoft/vscode-python@3fed49f:

Add Ruff to CI (microsoft/vscode-python#21739)

Add Ruff to (lint > action.yml) CI
#21738
--------------------
Commit message for microsoft/vscode-python@c490339:

Update version of npm package (microsoft/vscode-python#21765)


--------------------
Commit message for microsoft/vscode-python@8e0e59b:

Remove optionalDependencies from API npm package and document to install vscode types separately (microsoft/vscode-python#21764)

Closes It still leads to conflicts due to double installation of vscode
types when testing through the cases, removing vscode types as
dependencies altogether and documenting to install it separately
instead.
--------------------
Commit message for microsoft/vscode-python@cabdf39:

Use `optionalDependencies` instead of `peerDependencies` for `@vscode/python-extension` npm package (microsoft/vscode-python#21763)

Closes microsoft/vscode-python#21720
--------------------
Commit message for microsoft/vscode-python@dd20561:

revert due to buffer overflow on subprocess (microsoft/vscode-python#21762)

revert microsoft/vscode-python#21667 because it
causes buffer overflow in the python testing subprocess when larger
repos are used. Specifically seen on pytest discovery with >200 tests.
Revert to align with the stable release and put in a fix next week.
--------------------
Commit message for microsoft/vscode-python@40ff6e9:

Remove private Jupyter APIs from public API types (microsoft/vscode-python#21761)

For microsoft/vscode-jupyter#13986
--------------------
Commit message for microsoft/vscode-python@23353bb:

Improvements to `pythonTerminalEnvVarActivation` experiment (microsoft/vscode-python#21751)


--------------------
Commit message for microsoft/vscode-python@40bb62a:

fix spelling for get-pip.py (microsoft/vscode-python#21752)

fix spelling from get_pip to get-pip as advised.
--------------------
Commit message for microsoft/vscode-python@f454515:

Update release plan to document what to do with `main` during endgame week (microsoft/vscode-python#21743)


--------------------
Commit message for microsoft/vscode-python@dff25d4:

revert absolute test-ids (microsoft/vscode-python#21742)

seeing a substantial error where test discovery is broken. Reverting
this commit seems to be the temporary fix until I can diagnose the real
problem.

commit it is reverting:
microsoft/vscode-python#21682
--------------------
Commit message for microsoft/vscode-python@ca4dfd4:

update tests only on save with more files excluded (microsoft/vscode-python#21741)

fixes microsoft/vscode-python#21014 and
microsoft/vscode-python#21061
--------------------
Commit message for microsoft/vscode-python@84bbff9:

add cwd for debugging (microsoft/vscode-python#21668)

fixes microsoft/vscode-python#21648 (comment)
--------------------
Commit message for microsoft/vscode-python@ef16727:

Clean up smoke test requirement (microsoft/vscode-python#21729)

Cleaning up smoke test dependency: See if all Github action test pass
with removing the smoke test requirement file content.

Checked one by one, and came to see removing all doesn't seem to have
impact on the outcome of running smoke test.(Seems to have no difference
in smoke test result outcome when ran with "Run and Debug" in VS Code
with smoke-test option selected).

Also got rid of below, after checking smoke test correctly passing after
removal of smoke-test-requirement.txt content:
![Screenshot 2023-08-01 at 2 57 45
PM](https://github.com/microsoft/vscode-python/assets/62267334/45d404de-74dd-45a5-885b-71a25ef16ad7)

Resolve: microsoft/vscode-python#21496
--------------------
Commit message for microsoft/vscode-python@358635d:

Remove old and unused API for Jupyter Ext (microsoft/vscode-python#21731)

We have not used any of this API for a while now, hence its safe to
remove these.

Will be removing more soon.
--------------------
Commit message for microsoft/vscode-python@8f3d60b:

unittest discovery errors not displaying in test explorer (microsoft/vscode-python#21726)

saw an issue where if discovery failed there was no notice in the test
explorer for unittest. It was due to a different value for the new blank
value for the payload tests. fixes
microsoft/vscode-python#21725 and
microsoft/vscode-python#21688
--------------------
Commit message for microsoft/vscode-python@a6a8cb1:

Update main to next pre-release (microsoft/vscode-python#21728)


--------------------
Commit message for microsoft/vscode-python@4ab510d:

Update version for release candidate (microsoft/vscode-python#21727)


--------------------
Commit message for microsoft/vscode-python@3e7118f:

Update packages for Jedi and core python (microsoft/vscode-python#21710)


--------------------
Commit message for microsoft/vscode-python@237f82b:

Fix UUID and disposing to resolve race condition (microsoft/vscode-python#21667)

fixes microsoft/vscode-python#21599 and
microsoft/vscode-python#21507
--------------------
Commit message for microsoft/vscode-python@d9e368f:

add area-repl to issue label (microsoft/vscode-python#21718)

added area-repl as one of the issue label.
--------------------
Commit message for microsoft/vscode-python@ceecdb0:

Removing Jupyter Notebooks mentions from package.json (microsoft/vscode-python#21708)


--------------------
Commit message for microsoft/vscode-python@11a9f1d:

Remove unwanted Jupyter API (microsoft/vscode-python#21702)

Fixes microsoft/vscode-jupyter#13986
--------------------
Commit message for microsoft/vscode-python@efcc3d7:

Make test_ids relative to workspace path not root dir (microsoft/vscode-python#21682)

makes sure all testIds that are returned to the extension are relative
to the workspace (which will be the invocation directory) instead of to
the root. This will stop testIds for not being recognized when using a
config file or another parameter that changes the root directory during
pytest.

fixes microsoft/vscode-python#21640 and
microsoft/vscode-python#21637
--------------------
Commit message for microsoft/vscode-python@06d62aa:

Update homepage for Python API package (microsoft/vscode-python#21703)

For microsoft/vscode-python#21631
--------------------
Commit message for microsoft/vscode-python@83107cc:

Move "vscode" out of required dependencies for npm package (microsoft/vscode-python#21701)

Closes microsoft/vscode-python#21684
--------------------
Commit message for microsoft/vscode-python@d673004:

Convert JS-style typings to native TS in `@vscode/python-extension` (microsoft/vscode-python#21692)

Closes microsoft/vscode-python#21690
--------------------
Commit message for microsoft/vscode-python@8b9bca1:

Do not show "Select at workspace level" option if only one workspace folder is opened (microsoft/vscode-python#21689)

Closes microsoft/vscode-python#21220
--------------------
Commit message for microsoft/vscode-python@f536b74:

Edit issue-labels.yml, triage-info-needed.yml (microsoft/vscode-python#21685)

Add Anthony to issue-labels.yml and triage-info-needed.yml
--------------------
Commit message for microsoft/vscode-python@a42cb33:

Add new telemetry property to GPDR (microsoft/vscode-python#21683)

This property was added for tracking diagnostics we emit in Pylance.
--------------------
Commit message for microsoft/vscode-python@6af959d:

Dev Container Using MCR (microsoft/vscode-python#21675)

Dev container rewrite using MCR. 
Pyenv for installing and managing python versions. Fish also installed
as optional (able to view as shell option in codespaces). Also fixing
conda error. Takes care of: microsoft/vscode-python#21591 rewrite from: microsoft/vscode-python#21435 to adhere to
company policy.
--------------------
Commit message for microsoft/vscode-python@73a0e9d:

handle skip unittest at file without error (microsoft/vscode-python#21678)

fixes microsoft/vscode-python#21653
--------------------
Commit message for microsoft/vscode-python@9bcb82d:

Ensure `Run Python in dedicated terminal` uses `python.executeInFirDir` setting (microsoft/vscode-python#21681)


--------------------
Commit message for microsoft/vscode-python@713007f:

correct discovery on unittest skip at file level (microsoft/vscode-python#21665)

given a file called skip_test_file_node.py that has `raise
SkipTest(".....")` this should appear in the sidebar with no children.
The bug is that currently it shows a "unittest" node that gives "loader"
and other incorrect nodes below it.
--------------------
Commit message for microsoft/vscode-python@be334bd:

Do not resolve symbolic links in posix locator if they exceed the count limit (microsoft/vscode-python#21658)

Closes microsoft/vscode-python#21310

Fixes interpreter discovery running forever for non-Windows OS
--------------------
Commit message for microsoft/vscode-python@c256678:

Prevent posix paths locator from crashing (microsoft/vscode-python#21657)

For microsoft/vscode-python#21310
--------------------
Commit message for microsoft/vscode-python@81ae205:

Bring back feature to Run Python file in dedicated terminal (microsoft/vscode-python#21656)

Closes microsoft/vscode-python#21282
Closes microsoft/vscode-python#21420
Closes microsoft/vscode-python#21215
Reverts microsoft/vscode-python#21418
--------------------
Commit message for microsoft/vscode-python@c144200:

Modify .eslintrc to turn off any errors for declaration files (microsoft/vscode-python#21652)

For microsoft/vscode-python#21631
--------------------
Commit message for microsoft/vscode-python@f7125da:

Use correct `tsconfig.json` when generating npm package (microsoft/vscode-python#21651)

For microsoft/vscode-python#21631 

- Unset `removeComment` as that leads to declarations without docstrings
- Set to generate declarations
- Use updated typescript which results in cleaner declaration files
--------------------
Commit message for microsoft/vscode-python@2e8dc67:

Add extra logging regarding interpreter discovery (microsoft/vscode-python#21639)

For microsoft/vscode-python#21310
--------------------
Commit message for microsoft/vscode-python@fc1c391:

Compare global storage data using only `key` (microsoft/vscode-python#21636)

Closes microsoft/vscode-python#21635 by
applying the same fix as done in
microsoft/vscode-python#17627.

Lead-authored-by: Kartik Raj <[email protected]>
Co-authored-by: Daniel Imms <[email protected]>
Co-authored-by: Luciana Abud <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Courtney Webster <[email protected]>
Co-authored-by: Brett Cannon <[email protected]>
Co-authored-by: Erik De Bonte <[email protected]>
Co-authored-by: Rich Chiodo <[email protected]>
Co-authored-by: Don Jayamanne <[email protected]>
Co-authored-by: Anthony Kim <[email protected]>
Co-authored-by: Pete Farland <[email protected]>
Co-authored-by: Karthik Nadig <[email protected]>
Co-authored-by: Eleanor Boyd <[email protected]>
Signed-off-by: GitHub <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-testing bug Issue identified by VS Code Team member as probable bug verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants