Dr Jeff Jackson
Dr Jeff Jackson
Thanks, @UlisesGascon! PR #316 submitted.
I'm not sure why 13.x fails when 12.x succeeds, but I'm guessing that it has something to do with 13.x not having LTS. Anyway, I'm pretty sure that the 15.x...
The documentation has been updated to state that Node 14.x or higher is required for the --esm switch, and the --esm switch should now lead to an error exit for...
Am I correct in thinking that all of the PR test failures also occur when testing the master branch, or do I need to do some more work on the...
> type module and the node: prefix are both not required, and type module specifically should be avoided. ESM should be .mjs. Agreed, neither is required. The point of the...
@ljharb Thanks for the additional detail; that's helpful. Regarding your observation: > The `node:` prefix works in CJS also, and is not required in either; I'm not sure why it...
> @drjeffjackson do you have any plans to work on making the tests in the pipeline to pass? @joeyguerra: I believe that my PR code passes all of the tests...
> @drjeffjackson I pulled down the PR to investigate and found a bug in the `app.js.ejs` template. `createError` doesn't exist in the `--esm` -> "should generate a 404" test case....
> @drjeffjackson you're correct. I had removed lines 2 through 4 when troubleshooting the "http-errors not found" issue in the node 15 pipeline run. Ah, okay. So the issue is...
> @drjeffjackson the docs are written by folks with an agenda. I was on the modules working group, and i maintain module tooling, and I’m on the language committee, and...