Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes main by updating baselines #46160

Merged
merged 1 commit into from
Oct 1, 2021
Merged

Conversation

orta
Copy link
Contributor

@orta orta commented Oct 1, 2021

Introduced by #44813 not being fully rebased

@orta orta self-assigned this Oct 1, 2021
@@ -525,7 +525,7 @@ namespace Harness.Parallel.Host {
completeBar();
progressBars.disable();

const replayRunner = new Mocha.Runner(new Mocha.Suite(""), /*delay*/ false);
const replayRunner = new Mocha.Runner(new Mocha.Suite(""), { delay: false });
Copy link
Contributor Author

Choose a reason for hiding this comment

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

If you do a fresh npm install then you get runtime warnings about this

Copy link
Member

@weswigham weswigham Oct 1, 2021

Choose a reason for hiding this comment

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

This was deprecated officially in mocha 9, released in June, and has had the object form listed as an option for at least a few major versions longer - so the change seems fine to me~

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Oct 1, 2021
Copy link

@mrbrunelli mrbrunelli left a comment

Choose a reason for hiding this comment

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

👍

@andrewbranch andrewbranch merged commit 1a000c8 into microsoft:main Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants