-
Notifications
You must be signed in to change notification settings - Fork 10.2k
Closed
Labels
help wantedIssues marked as ideal for external contributorsIssues marked as ideal for external contributorstech-debtIssues and pull requests related to addressing technical debt or improving the codebaseIssues and pull requests related to addressing technical debt or improving the codebase
Description
After #6518 is reviewed and merged, we'll officially have no need to keep mocha and chai around. This means we should be able to remove these packages from the project:
@types/chai@types/chai-datetimechaichai-as-promisedchai-datetimemochamocha-junit-reportermocha-multi-reporters
As well as this test setup:
Lines 1 to 2 in 933efd9
| import { use } from 'chai' | |
| use(require('chai-datetime')) |
We can then drop this bit of documentation:
After doing that, you can check there are no outstanding issues by running these commands and ensuring the tests continue to pass:
$ yarn
$ yarn build:prod
$ yarn testReactions are currently unavailable
Metadata
Metadata
Assignees
Labels
help wantedIssues marked as ideal for external contributorsIssues marked as ideal for external contributorstech-debtIssues and pull requests related to addressing technical debt or improving the codebaseIssues and pull requests related to addressing technical debt or improving the codebase