Leo Singer
Leo Singer
Another similar bug: cyjake/ssh-config#74.
Blocked by https://github.com/codecov/codecov-action/issues/1293.
@dependabot rebase
NodeJS 18.x _does_ still support `require()`. The problem is that the AWS Lambda runtime for NodeJS 18.x now ships the AWS SDK v3, not the AWS SDK v2.
I think that #144 is poorly worded. Both Node.js 16.x _and_ 18.x support _both_ ESM (file extension: .mjs) _and_ CommonJS (file extension: .cjs) modules. The issue that this PR addresses...
No changes are needed regarding the module system.
If you copy the file and save it as `index.mjs` then it will fail, because the `.mjs` extension denotes an ECMAScript Module (ESM), and this package is written in CommonJS...
> Since this is specifically for AWS Lambda, where ESM is used by default with no package.json or any other configuration No, I don't think that's correct. In the absence...
> nice. I just was going to submit a PR for this exact thing. @lpsinger any reason you went with the v2 ses client rather than the v3? There is...
> also, when assigning `emailData`, did `Body.toString()` give back a string? I was having to use an async function called `transformToString` to get back the string. I pushed a possible...