search

ZedThree / clang-tidy-review

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

Path replace is broken #20

Closed iphydf closed 2 years ago

iphydf commented 2 years ago 
Replacing '/home/runner/work/c-toxcore/c-toxcor' with '/github/workspace'
...
LLVM ERROR: Cannot chdir into "/github/workspacee"!

https://github.com/TokTok/c-toxcore/runs/4472823526?check_suite_focus=true#step:6:165

This is our config file: https://github.com/TokTok/c-toxcore/blob/cf9a0b13418ef78cff6ac5c3d8dd1830360bab77/.github/workflows/clang-tidy-review.yml

kgfoundrydig commented 2 years ago

Yep, still broken: clang-tidy failed with return code 1 and error: LLVM ERROR: Cannot chdir into "/github/workspacet"!

My path ends with a t... it's getting included at the end of the local workspace directory.

Can we use an earlier version to avoid this?

ZedThree commented 2 years ago

Fixed by #21

ZedThree commented 2 years ago

In release 0.7.1