Skip to content

Conversation

karthiknadig
Copy link
Member

@karthiknadig karthiknadig commented Oct 21, 2019

No description provided.

Kartik Raj and others added 4 commits October 21, 2019 11:47
…pt is dispalyed (microsoft#7742)

* Added button to get more info related to inheritEnv setting when prompt is displayed

* Use aka.ms URL
* Don't prompt inheritEnv on windows
* Add verification for undefined workspace config
* Why did i commit that formatting change
* Disabling new debug adapter experiments
* Apply suggestions from code review for turning off debug adapter experiment
@karthiknadig karthiknadig added the no-changelog No news entry required label Oct 21, 2019
@codecov-io
Copy link

codecov-io commented Oct 21, 2019

Codecov Report

Merging #8129 into release will increase coverage by 0.01%.
The diff coverage is 92.3%.

Impacted file tree graph

@@             Coverage Diff             @@
##           release    #8129      +/-   ##
===========================================
+ Coverage    58.78%   58.79%   +0.01%     
===========================================
  Files          496      496              
  Lines        22126    22134       +8     
  Branches      3560     3562       +2     
===========================================
+ Hits         13006    13014       +8     
  Misses        8308     8308              
  Partials       812      812
Impacted Files Coverage Δ
src/client/telemetry/index.ts 86.13% <ø> (ø) ⬆️
src/client/common/utils/localize.ts 93.95% <100%> (+0.01%) ⬆️
...t/interpreter/virtualEnvs/condaInheritEnvPrompt.ts 98.24% <90.9%> (+0.24%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 48fe766...1f53413. Read the comment docs.

@karthiknadig karthiknadig merged commit 7ee8309 into microsoft:release Oct 21, 2019
DonJayamanne added a commit that referenced this pull request Oct 23, 2019
* release:
  Revert changes
  Mac smoke tests are not completing, but passing
  October point release 1 (#8156)
  Point release for Extension with data science changes (#8153)
  Cherry pick inhertEnv and disable debug adapter experiment into release (#8129)
  Preparing for release (#7828)
  Interactive Window and Native Editor react to font changes in VS Code settings (#7799)
  Adding both apache and BSD licenses for `packaging`  based on CELA recommendation. (#7763) (#7785)
  Turn on the native editor by default (#7777)
  Disable flaky smoke tests (#7776)
  Disable smoke tests against 2.7 (#7774)
  Fix liveshare guest hang issue (#7764) (#7772)
  Add missing PyPI packages to toml (#7749) (#7758)
  Ensure datascience smoke tests run only on python3 (#7755)
kimadeline pushed a commit that referenced this pull request Oct 24, 2019
* Ensure datascience smoke tests run only on python3 (#7755) 
* Add missing PyPI packages to toml (#7749) (#7758) 
* Fix liveshare guest hang issue (#7764) (#7772) 
* Disable smoke tests against 2.7 (#7774) 
* Disable flaky smoke tests (#7776) 
* Turn on the native editor by default (#7777) 
* Adding both apache and BSD licenses for `packaging` based on CELA recommendation. (#7763) (#7785)
* Interactive Window and Native Editor react to font changes in VS Code settings (#7799)
* Preparing for release (#7828) 
* Cherry pick inhertEnv and disable debug adapter experiment into release (#8129).
* Point release for Extension with data science changes (#8153)
* October point release 1 (#8156)
* Mac smoke tests are not completing, but passing
* Revert changes
* Merge from release into master
kimadeline added a commit that referenced this pull request Oct 25, 2019
* Ensure datascience smoke tests run only on python3 (#7755)

* Add missing PyPI packages to toml (#7749) (#7758)

* Fix liveshare guest hang issue (#7764) (#7772)

* initial changes

* update notebooks on server session change and add test

* await correctly

* Disable smoke tests against 2.7 (#7774)

* Disable flaky smoke tests (#7776)

* Disable smoke tests against 2.7
* Restore old smoke tests

* Turn on the native editor by default (#7777)

* Adding both apache and BSD licenses for `packaging`  based on CELA recommendation. (#7763) (#7785)

* Interactive Window and Native Editor react to font changes in VS Code settings (#7799)

* Make interactive window and native editor take
their fontSize and fontFamily from the settings in VS Code.

We get them in nativeEditor.tsx and in interactivePanel.tsx
to apply them as styles, but we also need to send them
as props to eventually assign them in the monaco editor.

* Moved style from react code to common.css

We now get the VS Code styles from the state of the
component, instead of computing a style from an element.

* Removed getting the font from nativeEditor and
interactivePanel, and put it in the mainStateController

* Made the interactiveWindow and the nativeEditor
react and change their fonts when the user changes them
in VS Code, instead of having to reload.

* Changed the font props to be an object instead of
two separate props to make it easier to maintain.

React Context was not used to avoid making the reuse of
components more difficult.

* Removed some unnecesary passing around of the
font prop

* removed unnecesary comments

* fixed a failing test

* Preparing for release (#7828)

* Allow the language server to choose its default caching mode (#7822)
* Preparing for release
* Update to change logs
* Grammar and typo fixes.

* Cherry pick inhertEnv and disable debug adapter experiment into release (#8129)

* Point release for Extension with data science changes (#8153)

* Expand variable explorer tests to work for native editor too. (#7699)

* Expand variable explorer tests to work for native editor too.

* Add comment about async fix for editor open

* Review feedback

* Allow connection to servers with no token and no password (#7708)

* Add capability to support current file path for notebook root (#7724)

* Add capability to support current file path for notebook root

* Put launch.json back

* Use system variables instead of creating separate resolver

* Fix mistake in config settings with passing in a uri instead of a string

* Make arguments more explicit

* Fix IS_WINDOWS to work on unit tests

* Better cancellations (#7714)

* Variable explorer and mime test fixes for tests (#7740)

* Change variable explorer tests to wait for variable explorer complete

* Fix native mime test

* Review feedback

* Try to make 2.7 tests pass

* Fix liveshare guest hang issue (#7764)

* initial changes

* update notebooks on server session change and add test

* await correctly

* Fix jupyter server startup hang when xeus-cling is installed (#7773)

* Make interactive window and native editor take their fontSize and fontFamily from the settings in VS Code. (#7746)

* Make interactive window and native editor take
their fontSize and fontFamily from the settings in VS Code.

We get them in nativeEditor.tsx and in interactivePanel.tsx
to apply them as styles, but we also need to send them
as props to eventually assign them in the monaco editor.

* Moved style from react code to common.css

We now get the VS Code styles from the state of the
component, instead of computing a style from an element.

* Removed getting the font from nativeEditor and
interactivePanel, and put it in the mainStateController

* Made the interactiveWindow and the nativeEditor
react and change their fonts when the user changes them
in VS Code, instead of having to reload.

* Changed the font props to be an object instead of
two separate props to make it easier to maintain.

React Context was not used to avoid making the reuse of
components more difficult.

* Removed some unnecesary passing around of the
font prop

* removed unnecesary comments

* fixed a failing test

* Fix a number of perf issues with large notebooks (#7778)

* First idea - load cells all at once.
Change updates to not update full cell
Change global properties to be per cell
Change cells to check for updates before rendering

* Fix delete to not select at the same time

* Make a pending state and a rendered state to use for rendering
Some perf fixes for monaco editor. Skip laying out parent
etc until necessary

* Refactor intellisense to not take so much time on load

* More delete and move fixes for intellisense

* Fix markdown not switching
Fix markdown editor background

* Fix some of the time on first render

* Add some more unit tests and fix intellisense issue

* Add news entry and fix variable explorer open

* Remove unnecessary state change

* Remove unnecessary merge

* Comment change

* Postpone widget parent changes

* Add new dependency

* Update some comments and names to be clearer

* Change the default cell marker (#7782)

* Change the default cell marker

* Fix test failures
Fix gather to use new comment
Fix formatting on code comments from nls

* Fix unit test too

* Fix gather to replace #%%

* Fix selection and focus not updating when clicking around in a notebook (#7806)

* Fix selection and focus not updating when clicking around in a notebook editor.

* Also fix add cell to put focus in the new cell

* More functional tests for the notebook editor (#7804)

* New tests

* Add some more functional tests for the notebook editor
Make sure when an error occurs on running a cell it's displayed to the user.

* Review feedback

* Add native command palette cell commands and recontext IW cell commands (#7807)

* Tests for keyboard shortcuts in native editor (#7797)

* Tests  for keyboard shortcuts
* Added some more tests
* Code review comments
* Fix merge issues

* Smoke tests does not depend on build stage (#7819)

* Smoke tests does not depend on build stage
* Faster CI

* Tests for undo using keyboard shortcut 'z' (#7824)

* Update @types/vscode (#7833)

* Update vscode dependency
* News entry

* Fix focus problems with add new cell buttons (#7827)

* Fix focus problems with add new cell buttons

* Get rid of unnecessary delay for async callbacks

* Use VSC API instead of hardcoding the uri resource vscode-resource (#7835)

* Update vscode dependency
* News entry
* Stop hardcoding 'vscode-resource' and use VSC api

* Remove jison dependency (#7863)

* Another fix for focus on add (#7883)

* Auto save native editor notebook (#7831)

* Fix ctrl+s to work in the notebook editor (#7919)

* Fix ctrl+s to work in the notebook editor

* Fix linting error

* More linter fixes

* Cherry pick commit b6545e3

* Prevent updates to cell text when stauts of cell has changed (#7973)

* Prevent updates to cell text when stauts of cell has changed
* Added comments

* When automatically opening the `Notebook Editor`, non file schemes (#7921)

* Ensure we support live share

* Copy *.trie files related to fontkit, as part of webpacking (#7978)

* Cherry pick saving fix for ipynb files

* new notebook command outside of extension (#7981)

* Hide the parameters list when editor doesn't have focus (#7987)

* Added comments

* Fix max output size to be passed down to rendering (#8017)

* Fix intellisense popping up in the wrong spot when first typing in a cell (#8018)

* Fix markdown disappearing after editing and hitting the escape key. (#8047)

* Fix markdown disappearing after editing and hitting the escape key.

* Put back Don's hack

* Fixes to allowing users to open a diff view for ipynb files (#8081)

* Fixes to allowing users to open a diff view
* Updated code review comments

* Ensure metadata for notebooks are always present (#8109)

* Ensure metadata for notebooks are always present
* Let storage run in background
* Refactor
* Address code review

* Do not auto save if notebook is untitled (#8095)

* Do not auto save if notebook is untitled
* Remove redundant code

* October point release 1 (#8156)

* New point release

* Mac smoke tests are not completing, but passing

* Revert changes

* Delete changelog items in point release

* Fix native editor merge changes

* Undo format on save changes + bad merge

* Remove commented out code
@lock lock bot locked as resolved and limited conversation to collaborators Oct 29, 2019
@karthiknadig karthiknadig deleted the release-a branch November 1, 2019 17:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants