• 
      

    Make CppLint optional and update documentation

    Review Request #10159 — Created Sept. 21, 2018 and submitted — Latest diff uploaded

    Information

    ReviewBot
    master
    55a997b...

    Reviewers

    Before this change, ReviewBot would try to install CppLint whether or not the user required it. This change removes CppLint from the required dependencies and creates an optional target 'all' that installs CppLint. This target can be extended in the future to include other dependencies.

    This change also updates the CppLint installation documentation to tell the user that they will need to manually install CppLint to use it.

    pip install -e ./bot does not install CppLint
    pip install -e ./bot[all] installs CppLint