Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix failing ipywidget tests #11132

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

Merged
merged 1 commit into from
Apr 13, 2020
Merged

Conversation

DonJayamanne
Copy link

For #11099

@DonJayamanne DonJayamanne added the no-changelog No news entry required label Apr 13, 2020
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov-io
Copy link

Codecov Report

Merging #11132 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11132      +/-   ##
==========================================
- Coverage   61.47%   61.44%   -0.03%     
==========================================
  Files         597      597              
  Lines       32801    32801              
  Branches     4640     4640              
==========================================
- Hits        20163    20154       -9     
- Misses      11622    11629       +7     
- Partials     1016     1018       +2     
Impacted Files Coverage Δ
src/datascience-ui/react-common/arePathsSame.ts 75.00% <0.00%> (-12.50%) ⬇️
src/client/common/utils/platform.ts 64.70% <0.00%> (-11.77%) ⬇️
...ence/ipywidgets/localWidgetScriptSourceProvider.ts 88.13% <0.00%> (-3.39%) ⬇️
src/client/linters/pydocstyle.ts 86.66% <0.00%> (-2.23%) ⬇️
src/client/datascience/debugLocationTracker.ts 76.56% <0.00%> (-1.57%) ⬇️
...client/datascience/kernel-launcher/kernelFinder.ts 75.70% <0.00%> (-0.94%) ⬇️
src/client/common/process/proc.ts 14.49% <0.00%> (-0.73%) ⬇️

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 56051b8...9295fb6. Read the comment docs.

@DonJayamanne DonJayamanne merged commit b556423 into microsoft:master Apr 13, 2020
@DonJayamanne DonJayamanne deleted the issue11099 branch April 13, 2020 20:28
DonJayamanne added a commit that referenced this pull request Apr 13, 2020
DonJayamanne added a commit that referenced this pull request Apr 14, 2020
* Load widget scripts from CDN and/or local python interpreter (#10987)

* Address sonar issues
* Fix linter
* Fixes
* Better way to pas array buffer
* Added comments
* Oops

* Copy widget scripts to extension folder (#11082)

* Copy widget scripts to extension folder
* Fix test

* Remove aml scripts from extension (#11085)

* Validate python when nb opened not on extension activates (#11087)

For #10893

* Support workspace images in markdown cells (#11086)

For #10893

* Fix functional tests for ipywidgets and download just chrome b… (#11093)

For #11092

playwright downloads other browsers, we need just one chrome (as this is closest to what VSCode is built upon)

* Added tests for k3d widget (#11133)

For #10799, #11096

* Fix failing ipywidget tests (#11132)

For #11099

* Update change log
@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
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