Szymon Łągiewka
|
41113599af
|
fix(refactor): prefix built-in node module imports
Since v5 relies on node >= 18, this is now possible (since v16, v14.18.0
[^1][^2]).
It's functionally irrelevant:
1. It's not required for CJS nor ESM (with a few exceptions [^3])
2. It has no performance promises
However, there are upsides to this approach:
1. It brings clear boundaries to what's a built-in and what's an
external dependency
2. It reduces the risk of importing unwanted deps where a built-in is
expected
3. It's slightly more interoperable with other JS runtimes that provide
node compatibility[^4], albeit only during development. Once imported
from npm, built-ins are assumed.
[^1]:https://nodejs.org/docs/latest-v22.x/api/modules.html#built-in-modules
[^2]:https://github.com/nodejs/node/pull/37246
[^3]:https://nodejs.org/api/modules.html#built-in-modules-with-mandatory-node-prefix
[^4]:https://docs.deno.com/runtime/fundamentals/node/#using-node's-built-in-modules
|
2025-01-10 11:53:44 -06:00 |
|
Douglas Christopher Wilson
|
a65913776d
|
tests: use strict mode
|
2022-02-09 01:07:08 -05:00 |
|
Douglas Christopher Wilson
|
c762b16f62
|
Improve error with invalid arguments to req.get()
fixes #2993
|
2016-06-03 00:07:34 -04:00 |
|
Jonathan Ong
|
643397ed21
|
remove unnecessary test/support/http
|
2014-03-05 22:06:14 -08:00 |
|
TJ Holowaychuk
|
0f5560eebd
|
updated tests to use "supertest"
|
2012-06-26 17:14:07 -07:00 |
|
TJ Holowaychuk
|
685eec0149
|
assertion for req.get() => undefined. Closes #1152
|
2012-05-30 14:25:43 -07:00 |
|
TJ Holowaychuk
|
85d6964874
|
Removed req.header(field)
|
2012-02-07 08:32:08 -08:00 |
|