uranusjr commented on code in PR #70812:
URL: https://github.com/apache/airflow/pull/70812#discussion_r3762623778


##########
ts-sdk/docs/tsconfig.json:
##########
@@ -0,0 +1,10 @@
+{
+  "//": "Docs-only tsconfig for the TypeDoc analysis pass. It inherits the 
SDK's compiler settings verbatim and roots the program at the public entry 
point only: TypeScript pulls in whatever that entry point transitively imports, 
but a glob like `../src/**/*.ts` would also root unreachable internal modules 
(e.g. the `cli/` bin, which needs the optional `esbuild` peer dependency) that 
the docs toolchain has no reason to install. `paths` below redirects 
`@msgpack/msgpack`, a real transitive dependency of the checked program 
(coordinator/frames.ts, reachable from the public startCoordinator export): 
Node module resolution walks up node_modules from the *importing file* 
(ts-sdk/src/coordinator/), never sideways into this package's own node_modules, 
so the bare specifier would otherwise never resolve here.",

Review Comment:
   Is wrapping possible here? This is very long.



-- 
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]

Reply via email to