Skip to content

Sync VSCode output Cell Output & add INotebookEditor/INotebookEditorProvider#11849

Merged
DonJayamanne merged 10 commits intomicrosoft:masterfrom
DonJayamanne:syncVSCodeOutput
May 18, 2020
Merged

Sync VSCode output Cell Output & add INotebookEditor/INotebookEditorProvider#11849
DonJayamanne merged 10 commits intomicrosoft:masterfrom
DonJayamanne:syncVSCodeOutput

Conversation

@DonJayamanne
Copy link

@DonJayamanne DonJayamanne commented May 16, 2020

For #10496

  • As we execute cell output ensure the VSCode output is kept upto date (Us to VSCode).
  • As VSCode makes changes we need to udpate our model (that is pending VSC api changes)
  • Create the INotebookEditor and INotebookEditorProvider (used all over in our extension for DS code).

@DonJayamanne DonJayamanne marked this pull request as ready for review May 16, 2020 00:24
@DonJayamanne DonJayamanne added the no-changelog No news entry required label May 16, 2020
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:

@codecov-io
Copy link

Codecov Report

Merging #11849 into master will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11849      +/-   ##
==========================================
- Coverage   60.45%   60.43%   -0.03%     
==========================================
  Files         631      631              
  Lines       34162    34166       +4     
  Branches     4799     4800       +1     
==========================================
- Hits        20653    20648       -5     
- Misses      12509    12516       +7     
- Partials     1000     1002       +2     
Impacted Files Coverage Δ
src/client/ioc/serviceManager.ts 40.90% <0.00%> (-4.10%) ⬇️
src/client/ioc/types.ts 100.00% <ø> (ø)
src/datascience-ui/react-common/arePathsSame.ts 75.00% <0.00%> (-12.50%) ⬇️
src/client/common/utils/platform.ts 64.70% <0.00%> (-5.89%) ⬇️
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%) ⬇️

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 3d7c03c...8f74de1. Read the comment docs.

public get onExecutedCode(): Event<string> {
return this.executedCode.event;
}
public notebook?: import('../types').INotebook | undefined;
Copy link
Member

Choose a reason for hiding this comment

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

Interesting, didn't know you could do this.

@DonJayamanne DonJayamanne merged commit 3300b83 into microsoft:master May 18, 2020
@DonJayamanne DonJayamanne deleted the syncVSCodeOutput branch May 18, 2020 15:18
@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 2 Code Smells

No Coverage information No Coverage information
2.0% 2.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants