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

Fix diagnostic position and clearance on document close #1699

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
868d389
Better handling of notebook diagnostics (#16529)
DonJayamanne Jun 21, 2021
a5ba414
Merge back from release to main (#16530)
karthiknadig Jun 21, 2021
67590fb
Lint files affected by the Jupyter work (#16462)
kimadeline Jun 22, 2021
422873d
Update flake8 default path value (#16540)
luabud Jun 23, 2021
e457bed
Ensure JediLSP restart command is registered only once. (#16551)
karthiknadig Jun 23, 2021
16497b6
Replace deprecated ExP methods with the newer ones (#16486)
kimadeline Jun 23, 2021
846eee8
Take Karthik out of the review queue
brettcannon Jun 24, 2021
fe85613
Remove SafeNotebookDocument API compat wrapper class (#16570)
joyceerhl Jun 28, 2021
b97942d
Add language server support for native interactive window (#16560)
joyceerhl Jun 29, 2021
767406d
Remove diagnostics on notebook/iw close (#16586)
rebornix Jul 2, 2021
8d84a29
Bump packaging from 20.9 to 21.0 (#16608)
dependabot[bot] Jul 7, 2021
452d11e
apply system variables for poetry path (#16607)
tonybaloney Jul 7, 2021
b4641ae
Disable insiders tests (#16626)
karthiknadig Jul 7, 2021
c60d03b
Update to latest jedi (#16624)
karthiknadig Jul 7, 2021
a0f0337
Sort available environments by assumed usefulness (#16559)
kimadeline Jul 8, 2021
9347aa4
Implement cases for pipenv and poetry module installers for the DataS…
tonybaloney Jul 8, 2021
4b40362
Correctly set `shellIdentificationSource`. (#16515)
intrigus-lgtm Jul 8, 2021
99d4bc3
16368 Only show last file/folder names on nodes in test explorer (#16…
bobwalker99 Jul 8, 2021
7d6a430
Fix diagnostics on cell and document close
rebornix Jul 8, 2021
4d53d81
fix format
rebornix Jul 8, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions .eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,6 @@ src/client/testing/display/main.ts
src/client/testing/display/picker.ts
src/client/testing/configuration.ts

src/client/common/serviceRegistry.ts
src/client/common/helpers.ts
src/client/common/net/browser.ts
src/client/common/net/fileDownloader.ts
Expand Down Expand Up @@ -517,8 +516,6 @@ src/client/common/application/clipboard.ts
src/client/common/application/workspace.ts
src/client/common/application/debugSessionTelemetry.ts
src/client/common/application/extensions.ts
src/client/common/application/types.ts
src/client/common/application/commandManager.ts
src/client/common/application/documentManager.ts
src/client/common/application/webPanels/webPanelProvider.ts
src/client/common/application/webPanels/webPanel.ts
Expand Down Expand Up @@ -646,7 +643,6 @@ src/startPage-ui/react-common/locReactSide.ts
src/startPage-ui/react-common/logger.ts
src/startPage-ui/react-common/postOffice.ts
src/startPage-ui/startPage/index.tsx
src/startPage-ui/startPage/startPage.tsx
src/test/startPage/mockCommandManager.ts
src/test/startPage/mockDocument.ts
src/test/startPage/mockDocumentManager.ts
Expand Down
1 change: 0 additions & 1 deletion .github/assign-reviewers.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ team:
# Comment yourself out when you will be unavailable for reviews for more than a
# couple of days.
reviewers:
- karthiknadig
- karrtikr
- kimadeline
- paulacamargo25
124 changes: 62 additions & 62 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -388,68 +388,68 @@ jobs:
with:
run: node --no-force-async-hooks-checks ./out/test/smokeTest.js

insider-tests:
name: Insider tests
if: github.repository == 'microsoft/vscode-python'
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
# We're not running CI on macOS for now because it's one less matrix entry to lower the number of runners used,
# macOS runners are expensive, and we assume that Ubuntu is enough to cover the UNIX case.
os: [ubuntu-latest]
python: [3.8] # For Jupyter.
steps:
- name: Checkout
uses: actions/checkout@v2.3.4

- name: Install Node
uses: actions/setup-node@v2.1.5
with:
node-version: ${{env.NODE_VERSION}}

- name: Use Python ${{matrix.python}}
uses: actions/setup-python@v2
with:
python-version: ${{matrix.python}}

- name: Install dependencies (npm ci)
run: npm ci --prefer-offline

- name: pip install system test requirements
run: |
python -m pip install --upgrade -r build/test-requirements.txt
python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade -r requirements.txt
python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade --pre debugpy
shell: bash

- name: pip install smoke test requirements
run: |
python -m pip install --upgrade -r build/smoke-test-requirements.txt
shell: bash

# Compile the test files.
- name: Prepare for insiders tests
run: npm run prePublish
shell: bash

- name: Set CI_PYTHON_PATH and CI_DISABLE_AUTO_SELECTION
run: |
echo "CI_PYTHON_PATH=python" >> $GITHUB_ENV
echo "CI_DISABLE_AUTO_SELECTION=1" >> $GITHUB_ENV
shell: bash

- name: Run insider tests
env:
DISPLAY: 10
INSTALL_JUPYTER_EXTENSION: true
INSTALL_PYLANCE_EXTENSION: true
VSC_PYTHON_CI_TEST_VSC_CHANNEL: insiders
TEST_FILES_SUFFIX: insiders.test
CODE_TESTS_WORKSPACE: ./src/testMultiRootWkspc/smokeTests
uses: GabrielBB/xvfb-action@v1.5
with:
run: node --no-force-async-hooks-checks ./out/test/standardTest.js
# insider-tests:
# name: Insider tests
# if: github.repository == 'microsoft/vscode-python'
# runs-on: ${{ matrix.os }}
# strategy:
# fail-fast: false
# matrix:
# # We're not running CI on macOS for now because it's one less matrix entry to lower the number of runners used,
# # macOS runners are expensive, and we assume that Ubuntu is enough to cover the UNIX case.
# os: [ubuntu-latest]
# python: [3.8] # For Jupyter.
# steps:
# - name: Checkout
# uses: actions/checkout@v2.3.4

# - name: Install Node
# uses: actions/setup-node@v2.1.5
# with:
# node-version: ${{env.NODE_VERSION}}

# - name: Use Python ${{matrix.python}}
# uses: actions/setup-python@v2
# with:
# python-version: ${{matrix.python}}

# - name: Install dependencies (npm ci)
# run: npm ci --prefer-offline

# - name: pip install system test requirements
# run: |
# python -m pip install --upgrade -r build/test-requirements.txt
# python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade -r requirements.txt
# python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade --pre debugpy
# shell: bash

# - name: pip install smoke test requirements
# run: |
# python -m pip install --upgrade -r build/smoke-test-requirements.txt
# shell: bash

# # Compile the test files.
# - name: Prepare for insiders tests
# run: npm run prePublish
# shell: bash

# - name: Set CI_PYTHON_PATH and CI_DISABLE_AUTO_SELECTION
# run: |
# echo "CI_PYTHON_PATH=python" >> $GITHUB_ENV
# echo "CI_DISABLE_AUTO_SELECTION=1" >> $GITHUB_ENV
# shell: bash

# - name: Run insider tests
# env:
# DISPLAY: 10
# INSTALL_JUPYTER_EXTENSION: true
# INSTALL_PYLANCE_EXTENSION: true
# VSC_PYTHON_CI_TEST_VSC_CHANNEL: insiders
# TEST_FILES_SUFFIX: insiders.test
# CODE_TESTS_WORKSPACE: ./src/testMultiRootWkspc/smokeTests
# uses: GabrielBB/xvfb-action@v1.5
# with:
# run: node --no-force-async-hooks-checks ./out/test/standardTest.js

upload:
name: Upload VSIX to Azure Blob Storage
Expand Down
126 changes: 63 additions & 63 deletions .github/workflows/pr-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -372,69 +372,69 @@ jobs:
with:
run: node --no-force-async-hooks-checks ./out/test/smokeTest.js

insider-tests:
name: Insider tests
# The value of runs-on is the OS of the current job (specified in the strategy matrix below) instead of being hardcoded.
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
# We're not running CI on macOS for now because it's one less matrix entry to lower the number of runners used,
# macOS runners are expensive, and we assume that Ubuntu is enough to cover the UNIX case.
os: [ubuntu-latest]
python: [3.8]
steps:
# Need the source to have the tests available.
- name: Checkout
uses: actions/checkout@v2.3.4

- name: Install Node
uses: actions/setup-node@v2.1.5
with:
node-version: ${{env.NODE_VERSION}}

- name: Use Python ${{matrix.python}}
uses: actions/setup-python@v2
with:
python-version: ${{matrix.python}}

- name: Install dependencies (npm ci)
run: npm ci --prefer-offline

- name: pip install system test requirements
run: |
python -m pip install --upgrade -r build/test-requirements.txt
python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade -r requirements.txt
python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade --pre debugpy
shell: bash

- name: pip install smoke test requirements
run: |
python -m pip install --upgrade -r build/smoke-test-requirements.txt
shell: bash

# Compile the test files.
- name: Prepare for insiders tests
run: npm run prePublish
shell: bash

- name: Set CI_PYTHON_PATH and CI_DISABLE_AUTO_SELECTION
run: |
echo "CI_PYTHON_PATH=python" >> $GITHUB_ENV
echo "CI_DISABLE_AUTO_SELECTION=1" >> $GITHUB_ENV
shell: bash

- name: Run insider tests
env:
DISPLAY: 10
INSTALL_JUPYTER_EXTENSION: true
INSTALL_PYLANCE_EXTENSION: true
VSC_PYTHON_CI_TEST_VSC_CHANNEL: insiders
TEST_FILES_SUFFIX: insiders.test
CODE_TESTS_WORKSPACE: ./src/testMultiRootWkspc/smokeTests
uses: GabrielBB/xvfb-action@v1.5
with:
run: node --no-force-async-hooks-checks ./out/test/standardTest.js
# insider-tests:
# name: Insider tests
# # The value of runs-on is the OS of the current job (specified in the strategy matrix below) instead of being hardcoded.
# runs-on: ${{ matrix.os }}
# strategy:
# fail-fast: false
# matrix:
# # We're not running CI on macOS for now because it's one less matrix entry to lower the number of runners used,
# # macOS runners are expensive, and we assume that Ubuntu is enough to cover the UNIX case.
# os: [ubuntu-latest]
# python: [3.8]
# steps:
# # Need the source to have the tests available.
# - name: Checkout
# uses: actions/checkout@v2.3.4

# - name: Install Node
# uses: actions/setup-node@v2.1.5
# with:
# node-version: ${{env.NODE_VERSION}}

# - name: Use Python ${{matrix.python}}
# uses: actions/setup-python@v2
# with:
# python-version: ${{matrix.python}}

# - name: Install dependencies (npm ci)
# run: npm ci --prefer-offline

# - name: pip install system test requirements
# run: |
# python -m pip install --upgrade -r build/test-requirements.txt
# python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade -r requirements.txt
# python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade --pre debugpy
# shell: bash

# - name: pip install smoke test requirements
# run: |
# python -m pip install --upgrade -r build/smoke-test-requirements.txt
# shell: bash

# # Compile the test files.
# - name: Prepare for insiders tests
# run: npm run prePublish
# shell: bash

# - name: Set CI_PYTHON_PATH and CI_DISABLE_AUTO_SELECTION
# run: |
# echo "CI_PYTHON_PATH=python" >> $GITHUB_ENV
# echo "CI_DISABLE_AUTO_SELECTION=1" >> $GITHUB_ENV
# shell: bash

# - name: Run insider tests
# env:
# DISPLAY: 10
# INSTALL_JUPYTER_EXTENSION: true
# INSTALL_PYLANCE_EXTENSION: true
# VSC_PYTHON_CI_TEST_VSC_CHANNEL: insiders
# TEST_FILES_SUFFIX: insiders.test
# CODE_TESTS_WORKSPACE: ./src/testMultiRootWkspc/smokeTests
# uses: GabrielBB/xvfb-action@v1.5
# with:
# run: node --no-force-async-hooks-checks ./out/test/standardTest.js

### Coverage run
coverage:
Expand Down
4 changes: 2 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Changelog

## 2021.6.0-rc (9 June 2021)
## 2021.6.0 (16 June 2021)

### Enhancements

Expand All @@ -20,7 +20,7 @@
([#16485](https://github.com/Microsoft/vscode-python/issues/16485))
1. Show `python.pythonPath` deprecation prompt when in `pythonDeprecatePythonPath` experiment.
([#16485](https://github.com/Microsoft/vscode-python/issues/16485))
1. Do not show safety prompt before autoselecting a workspace interpreter.
1. Do not show safety prompt before auto-selecting a workspace interpreter.
([#16485](https://github.com/Microsoft/vscode-python/issues/16485))
1. Assume workspace interpreters are safe to execute for discovery.
([#16485](https://github.com/Microsoft/vscode-python/issues/16485))
Expand Down
Loading