Skip to content

Run tests against insiders - #9872

Merged
Don Jayamanne (DonJayamanne) merged 3 commits into
microsoft:ds/custom_editorfrom
DonJayamanne:runTestsAgainstInsiders
Feb 3, 2020
Merged

Run tests against insiders#9872
Don Jayamanne (DonJayamanne) merged 3 commits into
microsoft:ds/custom_editorfrom
DonJayamanne:runTestsAgainstInsiders

Conversation

@DonJayamanne

@DonJayamanne Don Jayamanne (DonJayamanne) commented Feb 3, 2020

Copy link
Copy Markdown

No description provided.

@DonJayamanne Don Jayamanne (DonJayamanne) added the no-changelog No news entry required label Feb 3, 2020
@DonJayamanne

Don Jayamanne (DonJayamanne) commented Feb 3, 2020

Copy link
Copy Markdown
Author

Duh, CI hence close and reopen..

@DonJayamanne

Don Jayamanne (DonJayamanne) commented Feb 3, 2020

Copy link
Copy Markdown
Author

Ian Huff (@IanMatthewHuff) Rich Chiodo (@rchiodo) I'll remove the commented code from the redux logger. it was just adding too much stuff for me ..
Will remove once PR is approved. looks like all tests are going smoothly, functional tests and others as well.

@codecov-io

Codecov (codecov-io) commented Feb 3, 2020 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (ds/custom_editor@7ff3ed6). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@                 Coverage Diff                 @@
##             ds/custom_editor    #9872   +/-   ##
===================================================
  Coverage                    ?   60.64%           
===================================================
  Files                       ?      555           
  Lines                       ?    29726           
  Branches                    ?     4479           
===================================================
  Hits                        ?    18028           
  Misses                      ?    10697           
  Partials                    ?     1001

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 7ff3ed6...642b733. Read the comment docs.

rchiodo
Rich Chiodo (rchiodo) reviewed Feb 3, 2020
View reviewed changes
Comment thread src/datascience-ui/interactive-common/redux/store.ts Outdated
});
const loggerMiddleware = process.env.VSC_PYTHON_FORCE_LOGGING !== undefined || (process.env.NODE_ENV !== 'production' && !testMode) ? logger : undefined;

// const loggerMiddleware = process.env.VSC_PYTHON_FORCE_LOGGING !== undefined || (process.env.NODE_ENV !== 'production' && !testMode) ? logger : undefined;

@rchiodo Rich Chiodo (rchiodo) Feb 3, 2020

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Did you intend to remove this?

@rchiodo Rich Chiodo (rchiodo) Feb 3, 2020

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Never mind, saw your comment now.

@DonJayamanne
Don Jayamanne (DonJayamanne) merged commit bb95b58 into microsoft:ds/custom_editor Feb 3, 2020
@DonJayamanne
Don Jayamanne (DonJayamanne) deleted the runTestsAgainstInsiders branch February 3, 2020 21:34
@sonarqubecloud

sonarqubecloud Bot commented Feb 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

@lock lock Bot locked as resolved and limited conversation to collaborators Feb 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@rchiodo Rich Chiodo (rchiodo) rchiodo approved these changes
@IanMatthewHuff Ian Huff (IanMatthewHuff) IanMatthewHuff approved these changes
@DavidKutu David Kutugata (DavidKutu) Awaiting requested review from DavidKutu

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.

4 participants

@DonJayamanne @codecov-io @IanMatthewHuff @rchiodo