MetOffice / MetOffice/nemo-feedback

Use clang-tidy instead of cpplint to autolint

Open
#63 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
1
Forks
1
Avg merge
2d 15h
Merged PRs (30d)
1

Description

## Description

It would be nice if the linter would just automatically apply a style so that we never have to think about our cpp style outside of discussions of the linter configuration.

`clang-tidy` seems like the standard tool for this. It supports the same google style options as cpplint and purports to be able to automatically detect and fix issues. Linting can be switched off using comments in the code for specific blocks if necessary.

## Dependencies

* It isn't worth even attempting this until we get our new computing infrastructure and can easily access modern compilers.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by confirming that the new computing infrastructure and modern compilers are available, then investigate how clang-tidy can replace cpplint while supporting the stated Google style options. Done means linting can automatically apply the agreed C++ style and can be disabled for specific code blocks when necessary.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.