nealrichardson commented on PR #41576: URL: https://github.com/apache/arrow/pull/41576#issuecomment-2135234306
> Do we definitely still want/need that article? I am a big +1 to removing redundant docs/code, and given that it's buried in the developer docs and it's not likely there'll be a ton of new Acero functions, we could, like, just delete it? I'm cool with deleting it. You're right that it's from [another era](https://github.com/apache/arrow/pull/11915) in the package's development. And if someone is going to add more bindings, there's hundreds of examples to copy now. > Always a fan of UX changes like this, and I _love_ the usage of `→` to suggest a concrete action. Out of curiosity, is this something we're emulating from somewhere else or something you came up with on this PR @nealrichardson? I guess I came up with it. Looking at the options in cli (https://cli.r-lib.org/reference/cli_bullets.html), I wanted to reserve the `i` information ones for clarifying details, the others didn't seem appropriate, and, well, `arrow` just seemed like a logical choice for this package :) -- 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]
