-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update NPM dependencies and shift to ESM module, not CJS.
The new version of chai pulled in as part of the update requires ESM only, which forces the test case to use ESM, which then forces the app.js to also use ESM. These changes make the test suite work with both "npm test" and "yarn test", though note that currently I need to set the environment variable "UV_USE_IO_URING=0" to avoid an error about "text file busy". I discovered this trick from a Github issue comment [1]. Hopefully this also works on AWS Lambda but we'll see. [1]: nodejs/docker-node#1912 (comment)
- Loading branch information
1 parent
16540b0
commit d2648e8
Showing
4 changed files
with
78 additions
and
109 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters