-
Notifications
You must be signed in to change notification settings - Fork 2
feat(all): Upgrade Node.js and drop EoLs support #161
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 21.6.2 | ||
| 25.2.1 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,8 @@ | ||
| /* eslint-disable react-hooks/rules-of-hooks */ | ||
| import { usePlugin } from "@assertive-ts/core"; | ||
| import { SymbolPlugin } from "@examples/symbol-plugin"; | ||
| import { RootHookObject } from "mocha"; | ||
|
|
||
| export function mochaHooks(): RootHookObject { | ||
| export function mochaHooks(): Mocha.RootHookObject { | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i see we are calling this object from Mocha now and not importing it, but why was the import deleted?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @SbsCruz remember we're not calling calling an object here, it's just the type definition. We don't need to import it because mocha exposes the whole |
||
| return { | ||
| beforeAll() { | ||
| usePlugin(SymbolPlugin); | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
qq, what does the "--forbid-only" flag do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SbsCruz it causes the tests to fail if there's any
it.only(..)tests. The .only method helps a lot during development, but it can inadvertently exclude tests on CI. That's why we add it to the check script 🙂