cj-zhukov commented on PR #19750: URL: https://github.com/apache/datafusion/pull/19750#issuecomment-3799793562
@Jefffrey Thanks for the review! I’ve fixed the capitalization issues for group titles (e.g. Data IO, SQL Ops, UDF, etc.) by introducing explicit handling for common abbreviations instead of naive Title Case. For now, I kept the existing lightweight parser for extracting metadata from `main.rs` doc comments and just improved it where needed. As a follow-up PR, would it make sense to explore using a small parsing crate (e.g. `nom`) to make this logic more robust and easier to evolve, or is the preference to keep this part dependency-free? Happy to follow whatever direction makes the most sense here. -- 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]
