kentkwu commented on PR #390: URL: https://github.com/apache/arrow-js/pull/390#issuecomment-3987799116
> > It is not strictly necessary and removing it might make things cleaner. However I don't plan on including it in this PR. > > If we remove it, can we keep the exclude as-is? > > @domoritz Do you remember why you added `src/ipc/metadata/message.ts` to `exclude` in `typedoc.json` in [apache/arrow#13758](https://github.com/apache/arrow/pull/13758) ? @kou it seems cleaner to omit the exclude as currently this file is only included as a one-off (the others are directories). However we can still do both. Happy to refactor `declare module` in a separate PR. -- 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]
