Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-15248

[C++][Docs] Improve instructions for C++ formatting/linting

    XMLWordPrintableJSON

Details

    Description

      Code Style, Linting, and CI could use the following:

      • clang-format is implicitly described, but should be made an explicit requirement.
      • We should describe how to format C++ code with archery.
      • We should describe how to format/lint C++ code with the CMake build.
      • We should describe how Archery discovers the clang-format/cpplint executables.
      • We should describe how to use the GitHub Actions bot to try to format code.
      • If the wrong version of clang-format or clang-tidy is found, we should explicitly note that instead of just saying 'not found'
      • We should clarify that the exact version of these tools is needed
      • Link to the LLVM downloads since it may not be obvious where to get all these linter tools

      Attachments

        Issue Links

          Activity

            People

              lidavidm David Li
              lidavidm David Li
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 3h 40m
                  3h 40m