petern48 commented on PR #580: URL: https://github.com/apache/sedona-db/pull/580#issuecomment-3879369335
> (I do a lot of pushing to docs, R, and Python and do a lot of branch switching and my IDE is already running clippy as I do that) just out of curiosity, is the concern that adding it to pre-commit would make clippy run twice? i always thought your concern was that you didn't want clippy to run very often, but it sounds like it does? FWIW, my personal workflow is usually to install clippy in pre-commit and just use `git commit --no-verify` to skip it in most cases, other than before pushing. Though I'm also someone who writes a concise number of commits, so I don't need to type out `--no-verity` super often. I'm a fan of adding instructions to the contrib guide to optionally make it more automatic. -- 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]
