qa

Merge pull request #10 from andrsd/dim-detect #32

Sign in to view logs

Summary
Jobs
- c++ linter
Run details
- Usage
- Workflow file

Triggered via push January 9, 2024 20:51

andrsd

main

Status Failure

Total duration 27s

Artifacts –

qa.yml

on: push

Annotations

1 error, 10 warnings, and 1 notice

Process completed with exit code 1.

src/main.cpp:39:5 [cppcoreguidelines-pro-type-member-init]: src/main.cpp#L39

constructor does not initialize these fields: xmin, xmax, ymin, ymax, zmin, zmax

src/main.cpp:39:5 [modernize-use-equals-default]: src/main.cpp#L39

use '= default' to define a trivial default constructor

src/main.cpp:115:1 [modernize-use-trailing-return-type]: src/main.cpp#L115

use a trailing return type for this function

src/main.cpp:131:5 [readability-else-after-return]: src/main.cpp#L131

do not use 'else' after 'return'

src/main.cpp:131:9 [readability-braces-around-statements]: src/main.cpp#L131

statement should be inside braces

src/main.cpp:147:98 [readability-braces-around-statements]: src/main.cpp#L147

statement should be inside braces

src/main.cpp:149:99 [readability-braces-around-statements]: src/main.cpp#L149

statement should be inside braces

src/main.cpp:151:9 [readability-braces-around-statements]: src/main.cpp#L151

statement should be inside braces

src/main.cpp:159:1 [modernize-use-trailing-return-type]: src/main.cpp#L159

use a trailing return type for this function

src/main.cpp:159:21 [bugprone-easily-swappable-parameters]: src/main.cpp#L159

2 adjacent parameters of 'get_entities_by_dim' of similar type are easily swapped by mistake

Run clang-format on src/main.cpp: src/main.cpp#L1

File src/main.cpp does not conform to Custom style guidelines. (lines 360)