github-actions[bot] opened a new pull request, #75603:
URL: https://github.com/apache/airflow/pull/75603
* Fix airflow-ts-pack for CommonJS packages using Node built-ins
airflow-ts-pack outputs an ESM bundle, and an ESM file has no
`require`. esbuild turns a CommonJS dependency's `require("fs")` into
a shim that throws `Dynamic require of "fs" is not supported` unless a
`require` is in scope. So any Dag importing such a package (yaml, pg,
axios, @aws-sdk/client-s3, ...) failed to pack, and users cannot pass
esbuild options to work around it.
* Move the require banner comment onto the esbuild option
(cherry picked from commit 8b2ef80dcde15d32d1a0ef5cae2b87dc13a3a6af)
Co-authored-by: Andrew Chang <[email protected]>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
