Refactor: Move internals file in node folder to `/node/internal`
Move internals files like _net.ts , _errors , etc to node/internal .
Target:
- [x] _net.ts
- [x] _async_hooks.ts #2051
- [x] _buffer.js #2643
- [x] _dtrace.ts
- [x] _errors_test.ts, _errors.ts
- [x] _fixed_queue.ts https://github.com/denoland/deno_std/pull/2064
- [x] _idna.ts
- [x] _options.ts #2065
- [ ] _readline.js
- [ ] _stream.js
- [x] _timers.ts #1699
- [x]
_util/_debuglog,ts->internal/util/debuglog.ts
@getspooky Thank you for creating this issue!
Right. We should keep the file structure as close as possible to Node.js structure. The above change make sense to me.
BTW node/_core.js is something we invented on our own. Maybe we should keep it somewhere else?
Also deno_std/node/_buffer.js seems corresponding to node/lib/buffer.js. So let's just rename it to deno_std/node/buffer.js in this case.
Please carefully compare node.js lib/ directory and deno_std node/ directory before starting working on each item
@kt3k thanks for feedback, i will start working on it
Also _timers will conflicts with #1699. Please avoid it at first.
I updated the description, and crossed out some items (_core.ts _events.js _next_tick.ts _utils.ts) which don't seem existing in internal/
@kt3k, I'll do _readline.mjs. Is there any issue in essentially merging the .mjs and .d.ts files into a .ts file? I gather that .mjs is separated to make following the node version easier to follow.
@iuioiua I think there's no such issue. Do you think we should merge .mjs and .d.ts into .ts?
Looks these are all done now. Thanks for your efforts!