Skip to content

Commit

Permalink
Add developer guideline to use east const. (#14836)
Browse files Browse the repository at this point in the history
This PR documents the libcudf preference for "east const." Follow-up from #13491, #13492, #13493, #13494.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - David Wendt (https://github.com/davidwendt)
  - Nghia Truong (https://github.com/ttnghia)
  - Yunsong Wang (https://github.com/PointKernel)

URL: #14836
  • Loading branch information
bdice authored Jan 23, 2024
1 parent 0edeb00 commit bb02514
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions cpp/doxygen/developer_guide/DEVELOPER_GUIDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,10 @@ and we try to follow his rules: "No raw loops. No raw pointers. No raw synchroni
Additional style guidelines for libcudf code include:
* Prefer "east const", placing `const` after the type. This is not
automatically enforced by `clang-format` because the option
`QualifierAlignment: Right` has been observed to produce false negatives and
false positives.
* [NL.11: Make Literals
Readable](https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#nl11-make-literals-readable):
Decimal values should use integer separators every thousands place, like
Expand Down

0 comments on commit bb02514

Please sign in to comment.