Skip to content

GAUD-10607: update to Mocha 12 - #1077

Draft
dlockhart wants to merge 1 commit into
mainfrom
GAUD-10607/mocha-12
Draft

GAUD-10607: update to Mocha 12#1077
dlockhart wants to merge 1 commit into
mainfrom
GAUD-10607/mocha-12

Conversation

@dlockhart

Copy link
Copy Markdown
Member

This is going to fail until Sep 7 at 11:02AM when the npm proxy cooldown window expires, but wanted to get these PRs ready to go.

Comment thread package.json
"eslint-config-brightspace": "^4",
"globals": "^17",
"mocha": "^11",
"mocha": "^12",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I was a little surprised that this was just a devDependency, but because we rely on @open-wc/testing and @web/test-runner to run the tests, it seems OK? They actually don't have a dependency on Mocha either, which is a bit strange.

Comment thread package-lock.json

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll re-generate this again on Monday as it picked up some other stuff that's too new.

Comment thread package.json
"@web/test-runner-playwright": "^1",
"command-line-args": "^6",
"command-line-usage": "^7",
"d2l-test-reporting": "^6",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This has a direct dependency on mocha, so we'll want to wait for it to update & release before releasing this.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant