c-taylor opened a new issue, #11754:
URL: https://github.com/apache/trafficserver/issues/11754

   I encountered a few issues when setting up the tools for pre-commit hooks on 
a recent brew:
   
   * 'Modern' brew python does not have 'pip' only 'pip3'
   * virtualenv and other yapf tools scripts hard require pip
   * Scripts use python3 by name, but not pip3 (seems inconsistent)
   * repo yapf is from 2021
   
   Additionally:
   pip installs make me uncomfortable.
   Maybe consider using system yapf/clang-format and publishing config rather 
than version locking?
   
   Reproduction:
   Remove all of your yapf/clang tooling in your repo
   Try to recreate it


-- 
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: issues-unsubscr...@trafficserver.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to