ZedThree / clang-tidy-review

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

Get username from PR to use in `google-readability-todo` #106

Closed bwrsandman closed 10 months ago

bwrsandman commented 10 months ago

Improves the recommendation for google-readability-todo by providing a proper name which would previously be TODO(docker) or TODO(unknown).

If for whatever reason, the name cannot be found, it will default to TODO(your name here).

Closes #102 without the need for a new parameter.

Note that I think using env in subprocess also kill all env leaking into the call for subprocess.run. Hopefully no one relies on that.