dentiny commented on PR #24351: URL: https://github.com/apache/datafusion/pull/24351#issuecomment-5289613831
> But it's not detecting those dependencies, perhaps we should update the CI first. Hi thanks for the quick reply! I double confirmed with my agent, it's not Machete -- Machete considers datafusion-catalog and datafusion-execution used because their names appeared in imports, even though they only re-exported types from narrower crates. The prompt I gave is ``` Find unused and misplaced dependencies in the repo, - we could use tools like Machete - check whether we could put test / benchmark only dependencies out of production ones ``` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
