Skip to content

Bump plotly.js-dist at v1.55.1#13774

Merged
rchiodo merged 1 commit into
microsoft:masterfrom
archmoj:bump-plotly.js-v1.55.1
Sep 3, 2020
Merged

Bump plotly.js-dist at v1.55.1#13774
rchiodo merged 1 commit into
microsoft:masterfrom
archmoj:bump-plotly.js-v1.55.1

Conversation

@archmoj

@archmoj archmoj commented Sep 3, 2020 •
edited
Loading

Copy link
Copy Markdown

Bumping plotly.js-dist module now that plotly.js v1.55.1 is out 🎉

  • package-lock.json has been regenerated by running npm install (if dependencies have changed).

Looks like PR #12609.
Also FYI a PR is submitted to nteract/outputs#17.

@rchiodo
cc: @nicolaskruchten

@ghost

ghost commented Sep 3, 2020 •
edited by ghost
Loading

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@sonarqubecloud

sonarqubecloud Bot commented Sep 3, 2020

Copy link
Copy Markdown

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
No Duplication information No Duplication information

@codecov-commenter

codecov-commenter commented Sep 3, 2020 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #13774      +/-   ##
==========================================
- Coverage   59.87%   59.85%   -0.03%     
==========================================
  Files         678      678              
  Lines       37871    37867       -4     
  Branches     5459     5458       -1     
==========================================
- Hits        22676    22666      -10     
- Misses      14023    14026       +3     
- Partials     1172     1175       +3
Impacted Files Coverage Δ
src/client/datascience/crossProcessLock.ts 79.41% <0.00%> (-11.77%) ⬇️
src/client/common/utils/platform.ts 56.00% <0.00%> (-8.00%) ⬇️
src/client/linters/pydocstyle.ts 86.66% <0.00%> (-2.23%) ⬇️
src/client/datascience/debugLocationTracker.ts 76.56% <0.00%> (-1.57%) ⬇️
src/client/common/process/proc.ts 14.49% <0.00%> (-0.73%) ⬇️
src/client/datascience/constants.ts 99.78% <0.00%> (-0.01%) ⬇️
...rc/client/datascience/commands/notebookCommands.ts 19.51% <0.00%> (+1.33%) ⬆️

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 687363d...694d48e. Read the comment docs.

@rchiodo rchiodo added the no-changelog No news entry required label Sep 3, 2020
@rchiodo
rchiodo merged commit 0b46678 into microsoft:master Sep 3, 2020
@archmoj
archmoj deleted the bump-plotly.js-v1.55.1 branch September 3, 2020 18:37
@archmoj

archmoj commented Sep 3, 2020

Copy link
Copy Markdown
Author

@rchiodo Thanks very much for the review!

@archmoj archmoj mentioned this pull request Sep 8, 2020
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rchiodo rchiodo rchiodo approved these changes

Assignees

No one assigned

Labels

no-changelog No news entry required

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@archmoj @codecov-commenter @rchiodo