austin3dickey commented on PR #41031: URL: https://github.com/apache/arrow/pull/41031#issuecomment-2040022164
>Where else do we call perf from? Could we just change all the calls to run yarn perf / npm run perf? As far as I know, everything that runs benchmarks just uses `yarn perf`. >Now we have the same command in two places If this is a problem, perhaps we should remove the shebang then? The trouble with the shebang as written is that using `/usr/bin/env -S` is non-portable. So if we're using `yarn perf` everywhere, it should be fine to remove the shebang. -- 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]
