PointCloudLibrary / PointCloudLibrary/pcl

Add Dockerfile (and image) for clang-tidy CI

Open
#4,641 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

effort: medium good first issue kind: todo module: ci
Dominant language
C++
Stars
11.1k
Forks
4.7k
Avg merge
4d 10h
Merged PRs (30d)
6

Description

Image can be based off the pointcloudlibrary/env and add it and clang-tidy (version dependent on the github workflow)

IIRC, doesn't clang-tidy require all dependencies to be installed?

Ahh, that might be. I'm not that familiar with clang/tidy.

I can see Cmake is run as well, which will fail with missing dependencies.

Originally posted by @larshg in https://github.com/PointCloudLibrary/pcl/pull/4636#issuecomment-795061118

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by examining the existing GitHub workflow that runs clang-tidy and the pointcloudlibrary/env image setup. Check which CMake and project dependencies that workflow requires, then define the Docker image so the workflow can run successfully with the required clang-tidy version. Done means the image builds and the clang-tidy CI job completes.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake, cpp, dockerfile, github-actions
Domain
build-system, ci-cd, devops
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.