Skip to content

Conversation

DonJayamanne
Copy link

For #7221

@DonJayamanne DonJayamanne requested a review from rchiodo December 20, 2019 00:44
@DonJayamanne DonJayamanne added the no-changelog No news entry required label Dec 20, 2019
@codecov-io
Copy link

codecov-io commented Dec 20, 2019

Codecov Report

Merging #9234 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9234      +/-   ##
==========================================
- Coverage    57.5%   57.48%   -0.03%     
==========================================
  Files         551      551              
  Lines       29432    29445      +13     
  Branches     4460     4462       +2     
==========================================
  Hits        16925    16925              
- Misses      11561    11574      +13     
  Partials      946      946
Impacted Files Coverage Δ
...rc/client/common/application/webPanels/webPanel.ts 14.28% <ø> (ø) ⬆️
...ent/common/application/webPanels/webPanelServer.ts 77.94% <ø> (ø) ⬆️
src/datascience-ui/interactive-common/handlers.ts 4.54% <0%> (-6.57%) ⬇️

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 477519e...35f64d9. Read the comment docs.

Copy link

@rchiodo rchiodo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@DonJayamanne DonJayamanne merged commit e4631cf into microsoft:master Dec 20, 2019
rchiodo pushed a commit that referenced this pull request Dec 20, 2019
* Allow downloading plotly images

* Oops
@lock lock bot locked as resolved and limited conversation to collaborators Dec 27, 2019
@DonJayamanne DonJayamanne deleted the fixPlotly branch April 30, 2020 17:02
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.

3 participants