2bndy5
commented
2 years ago Thanks for the link to the actual repo/workflow, that is most helpful! With that said, your extensive CMake presets are a bit difficult to follow for such a simple test.
There may be a problem with docker file system not seeing fmt/format.h in the runner's file system.
If you need to use a custom env (typically for non-std libs or cross-compiling), then I'd recommend trying the windows example workflow in the readme. You don't need to use windows, you just need to understand how to use this action's source as a python installable package. I put together an example based on the snippet you posted in the OP:
on:
pull_request:
types: [opened, reopened] # let PR-synchronize events be handled by push events
push:
jobs:
cpp-linter:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
- name: Install clang-tools
uses: KyleMayes/install-llvm-action@v1
with:
version: 14
# specifying an install path is not required for ubuntu
# directory: ${{ runner.temp }}/llvm
- name: Install linter python package
run: python3 -m pip install git+https://github.com/cpp-linter/cpp-linter-action@v1
- name: run linter as a python package
id: linter
run: cpp-linter --version=14 --style=file --tidy-checks='' --database=build/ci-linux-clang
- name: Fail fast?!
if: steps.linter.outputs.checks-failed > 0
run: echo "Some files failed the linting checks!"
# for actual deployment
# run: exit 1
ibis-hdl
BurningEnlightenment
I see. Thanks for answer. Thanks for this action and your time to support!
Meanwhile, I run into another issue with regard to clang tidy's database (I'm not sure to open another issue - it's at yours). My action looks like
where
CMakePresets.jsoncontainsThe action's log shows that it fails:
Otherwise, running VS Code's devcontainer shows relative path seems to work:
I haven't the experience with clang-tidy using CLI arguments and python, so my guess is that's there is a wrong path (otherwise not very likely) :(
If relevant, the action is here
Originally posted by @ibis-hdl in https://github.com/cpp-linter/cpp-linter-action/issues/64#issuecomment-1154250055