Git Product home page Git Product logo

Comments (5)

2bndy5 avatar 2bndy5 commented on June 15, 2024

Reading the workflow's log that you linked in the discussion, it looks like clang-tidy-14 was able to be called and returned a result. Even if that result is undesirable, the fact that the clang-tidy-14 executable was found (& invoked) tells me that it isn't an installation problem.

Have you tried using a compilation database (highly recommended)? This is a rather strict sticking point for clang-tidy (despite the version used).

from cpp-linter-action.

shenxianpeng avatar shenxianpeng commented on June 15, 2024

I have updated the docker image clang-tools:all and I can not reproduce this issue anymore.

Currently, this is a temporary solution: just copy the missing *.h files to /usr/include, because keeping llvm-13 and llvm-14 folders in the image will extremely increase the size of the image and cause action to slow down.

from cpp-linter-action.

2bndy5 avatar 2bndy5 commented on June 15, 2024

keeping llvm-13 and llvm-14 folders in the image will extremely increase the size of the image and cause action to slow down.

This is why I've been trying to use the action's src code as an installable python package. The docker image isn't needed if executing the action as a python package, but it does require the end-user to consciously install the desired version of clang-tidy & clang-format (github's actions/cache action can further reduce the setup-time in this scenario). Clearly, this will also require some finesse with the --version arg though.

from cpp-linter-action.

shenxianpeng avatar shenxianpeng commented on June 15, 2024

Currently, this is a temporary solution: just copy the missing *.h files to /usr/include

UPDATE: Permanent solution has been done and the docker image has been updated. See cpp-linter/clang-tools-docker#25

from cpp-linter-action.

shenxianpeng avatar shenxianpeng commented on June 15, 2024

This issue has been fixed in the latest docker image xianpengshen/clang-tools:all, I'm going to close it.
Please feel free to reopen it in case of any problems.

from cpp-linter-action.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.