search

voorkant / voorkant-core

https://voorkant.org/
MIT License
2 stars 3 forks source link

clang-tidy: naming conventions #52

Open Habbie opened 5 months ago

Habbie commented 5 months ago

we should enforce naming conventions in clang-tidy. A lot of our methods currently start with an uppercase letter (because @cyclops1982 also types a lot of Go), and others do not. I prefer lowercase there, but mostly I prefer consistency. @cyclops1982 also recently asked me how to name constants, and we should have a standard for that too.

cyclops1982 commented 5 months ago

PR #54 is a start.

cyclops1982 commented 2 months ago

We fixed this in one of the PR #60

Habbie commented 2 months ago

reopened: CI does not enforce it, and it should