Closed johnkerl closed 3 weeks ago
While working on #2407 / [sc-51048] and #2406.
This was fully automated via
clang-format -i apis/r/src/[a-z]*.cpp
with the exception of a couple fix-ups on //'-style comments.
//'
Note that we enitrely avoid apis/r/src/RcppExports.cpp which is auto-generated.
apis/r/src/RcppExports.cpp
On this PR I just manually ran the formatter on the relevant source files. On a follow-up PR I'll stop excluding these files from the format-checker which we run on the rest of our libtiledbsoma and pybind11 C++ source: https://github.com/single-cell-data/TileDB-SOMA/blob/1.14.1/.github/workflows/python-ci-single.yml#L62-L64
libtiledbsoma
pybind11
Also note that we should move the clang-format checker from python-ci-single.yml to another C++-specific YAML file.
clang-format
python-ci-single.yml
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 89.71%. Comparing base (fd1537f) to head (0fd2de9).
fd1537f
0fd2de9
While working on #2407 / [sc-51048] and #2406.
This was fully automated via
with the exception of a couple fix-ups on
//'
-style comments.Note that we enitrely avoid
apis/r/src/RcppExports.cpp
which is auto-generated.On this PR I just manually ran the formatter on the relevant source files. On a follow-up PR I'll stop excluding these files from the format-checker which we run on the rest of our
libtiledbsoma
andpybind11
C++ source: https://github.com/single-cell-data/TileDB-SOMA/blob/1.14.1/.github/workflows/python-ci-single.yml#L62-L64Also note that we should move the
clang-format
checker frompython-ci-single.yml
to another C++-specific YAML file.