Skip to content

Add code gathering for single cell#6803

Merged
joyceerhl merged 127 commits into
microsoft:masterfrom
joyceerhl:t-huer/gatherCode
Aug 1, 2019
Merged

Add code gathering for single cell#6803
joyceerhl merged 127 commits into
microsoft:masterfrom
joyceerhl:t-huer/gatherCode

Conversation

@joyceerhl
Copy link
Copy Markdown

@joyceerhl joyceerhl commented Jul 31, 2019

For #6810

@joyceerhl joyceerhl requested a review from rchiodo July 31, 2019 19:22
Comment thread build/ci/postInstall.js
Comment thread src/client/datascience/interactive-window/interactiveWindow.ts Outdated
Comment thread src/client/datascience/interactive-window/interactiveWindow.ts Outdated
Comment thread tsconfig.json Outdated
Comment thread src/client/datascience/gather/gather.ts Outdated
Comment thread src/client/datascience/interactive-window/interactiveWindow.ts Outdated
Comment thread news/1 Enhancements/6803.md
@rchiodo
Copy link
Copy Markdown

rchiodo commented Aug 1, 2019

It looks like the functional tests aren't working either. You might have to add more things to our document mocking in the reactHelpers.ts

@rchiodo
Copy link
Copy Markdown

rchiodo commented Aug 1, 2019

You can run them locally with 'npm run test:functional'. That should repro the bug

Comment thread package-lock.json
@joyceerhl
Copy link
Copy Markdown
Author

It looks like the functional tests aren't working either. You might have to add more things to our document mocking in the reactHelpers.ts

Yep fixed them here and here. Probably related to palantir/blueprint#397?

@joyceerhl joyceerhl merged commit 6d00a3e into microsoft:master Aug 1, 2019
@lock lock Bot locked as resolved and limited conversation to collaborators Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants