boostorg / boostorg/ublas

Include some Clang-Tidy checks

Open
#112 0 comments 0 reactions 2 assignees Claimed by @amitsingh19975 View on GitHub
enhancement
Dominant language
C++
Stars
122
Forks
151
PR merge metrics
No merged PRs in 30d

Description

The configuration [file](https://github.com/boostorg/ublas/blob/develop/.clang-tidy) excludes clang-tidy checks
```bash
google-readability-braces-around-statements
google-explicit-constructor
hicpp-vararg
cppcoreguidelines-pro-type-vararg
```
We should include them.
- `google-readability-braces-around-statements` requires simple refactoring
- `google-explicit-constructor` needs significant rework also in the test files and results in API change
- `hicpp-vararg` needs usage of the boost lightweight unit test instead of the boost-test

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.