codecov[bot] commented 4 years ago

Codecov Report

Merging #16 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   25.00%   25.00%           
=======================================
  Files           2        2           
  Lines           4        4           
=======================================
  Hits            1        1           
  Misses          3        3

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 92edf2c...a836d61. Read the comment docs.

bburda commented 4 years ago

Please add also readability check in clang-tidy config for following keys:

key: readability-identifier-naming.TypedefCase
key: readability-identifier-naming.StructCase
key: readability-identifier-naming.ClassCase
key: readability-identifier-naming.VariableCase
key: readability-identifier-naming.ParameterCase
key: readability-identifier-naming.FunctionCase
key: readability-identifier-naming.NamespaceCase
key: readability-identifier-naming.GlobalConstantCase
key: readability-identifier-naming.PrivateMemberSuffix

with values proper for Autosar guidelines: https://www.autosar.org/fileadmin/user_upload/standards/adaptive/19-11/AUTOSAR_RS_CPP14Guidelines.pdf

It would be great if you manage to add even more Autosar Guidelines checks :)

hliberacki commented 4 years ago

@bburda Jerzy has been deleted. This is dangling

bburda commented 4 years ago

The author deleted account. Closing.

UmlautSoftwareDevelopmentAccount / AP-Functional-Cluster-Template

Added clang-tidy step to travis file configuration #16

Codecov Report