search

ZedThree / clang-tidy-review

Create a pull request review based on clang-tidy warnings
MIT License
88 stars 44 forks source link

Add clang-tidy 16 #98

Closed bwrsandman closed 11 months ago

bwrsandman commented 11 months ago

Switch to ubuntu lunar. Versions 11 and 12 are not available. In this version, installing pip packages outside of a venv are disabled by default.

Fixes #88

ZedThree commented 11 months ago

In this version, installing pip packages outside of a venv are disabled by default.

This worries me a little bit as people might be installing pip packages for their build. I guess we can wait to see if anyone screams, and then fix it, either by remove the magic file, adding something to the pip config file, or just creating and enabling a venv by default.

ZedThree commented 11 months ago

Please could you also update the readme and action.yml with the available versions?

bwrsandman commented 11 months ago

I thought I did. Guess I forgot to push