add a workflow to check clang-format (#79)

Author: bkietz

.github/workflows/format.yaml

@@ -0,0 +1,24 @@
+on:
+  push:
+    branches: master
+  pull_request:
+    branches:
+      - master
+
+name: format_check
+
+jobs:
+  format_check:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Install ClangFormat
+        run: sudo apt-get install -y clang-format-10
+
+      - name: Run ClangFormat
+        run: make format clang_format=clang-format-10
+
+      - name: Check for a non-empty diff
+        run: git diff-files -U --exit-code
+

Makefile

@@ -10,3 +10,11 @@ test: all
 
 clean:
 	@Rscript -e 'devtools::clean_dll()'
+
+clang_format=`which clang-format`
+format: $(shell find . -name *.hpp) $(shell find . -name *.cpp)
+ifeq ($(findstring version 10,$(shell ${clang_format} --version 2>/dev/null)),)
+	@echo "clang-format 10 is required"
+else
+	@${clang_format} -i $?
+endif

cpp11test/src/test-list.cpp

@@ -59,9 +59,7 @@ context("list-C++") {
   }
 
   test_that("list::iterator uses VECTOR_ELT") {
-    cpp11::writable::list x({
-      cpp11::writable::integers({1, 2})
-    });
+    cpp11::writable::list x({cpp11::writable::integers({1, 2})});
     cpp11::integers first(*x.begin());
     expect_true(first[0] == 1);
     expect_true(first[1] == 2);

cpp11test/src/test-list_of.cpp

@@ -1,15 +1,15 @@
 #include <testthat.h>
-#include "cpp11/strings.hpp"
 #include "cpp11/doubles.hpp"
 #include "cpp11/list.hpp"
 #include "cpp11/list_of.hpp"
+#include "cpp11/strings.hpp"
 
 context("list_of-C++") {
   test_that("list_of works") {
     using namespace cpp11::literals;
 
     cpp11::writable::list x({"x"_nm = cpp11::writable::doubles({1., 2., 3.}),
-                               "y"_nm = cpp11::writable::doubles({4., 5., 6.})});
+                             "y"_nm = cpp11::writable::doubles({4., 5., 6.})});
 
     cpp11::list_of<cpp11::doubles> res(x);
 
@@ -24,7 +24,7 @@ context("list_of-C++") {
     using namespace cpp11::literals;
 
     cpp11::writable::list x({"x"_nm = cpp11::writable::doubles({1., 2., 3.}),
-                               "y"_nm = cpp11::writable::doubles({4., 5., 6.})});
+                             "y"_nm = cpp11::writable::doubles({4., 5., 6.})});
 
     cpp11::writable::list_of<cpp11::writable::doubles> res(x);
 

inst/include/cpp11.hpp

@@ -16,8 +16,8 @@
 #include "cpp11/matrix.hpp"
 #include "cpp11/named_arg.hpp"
 #include "cpp11/protect.hpp"
+#include "cpp11/r_string.hpp"
+#include "cpp11/r_vector.hpp"
 #include "cpp11/raws.hpp"
 #include "cpp11/sexp.hpp"
-#include "cpp11/r_string.hpp"
 #include "cpp11/strings.hpp"
-#include "cpp11/r_vector.hpp"

inst/include/cpp11/attribute_proxy.hpp

@@ -1,7 +1,8 @@
 #pragma once
 
-#include <initializer_list>   // for initializer_list
-#include <string>             // for string, basic_string
+#include <initializer_list>  // for initializer_list
+#include <string>            // for string, basic_string
+
 #include "cpp11/R.hpp"        // for SEXP, SEXPREC, Rf_install, PROTECT, Rf_...
 #include "cpp11/as.hpp"       // for as_sexp
 #include "cpp11/protect.hpp"  // for protect, safe, protect::function

inst/include/cpp11/data_frame.hpp

@@ -1,17 +1,17 @@
 #pragma once
 
-#include <cstdlib>                    // for abs
-#include <initializer_list>           // for initializer_list
-#include <string>                     // for string, basic_string
-#include <utility>                    // for move
+#include <cstdlib>  // for abs
+#include <cstdlib>
+#include <initializer_list>  // for initializer_list
+#include <string>            // for string, basic_string
+#include <utility>           // for move
+
 #include "R_ext/Arith.h"              // for NA_INTEGER
 #include "cpp11/R.hpp"                // for Rf_xlength, SEXP, SEXPREC, INTEGER
 #include "cpp11/attribute_proxy.hpp"  // for attribute_proxy
 #include "cpp11/list.hpp"             // for list, r_vector<>::r_vector, r_v...
 #include "cpp11/r_vector.hpp"         // for r_vector
 
-#include <cstdlib>
-
 namespace cpp11 {
 
 class named_arg;

inst/include/cpp11/doubles.hpp

@@ -1,8 +1,9 @@
 #pragma once
 
-#include <algorithm>            // for min
-#include <array>                // for array
-#include <initializer_list>     // for initializer_list
+#include <algorithm>         // for min
+#include <array>             // for array
+#include <initializer_list>  // for initializer_list
+
 #include "R_ext/Arith.h"        // for ISNA
 #include "cpp11/R.hpp"          // for SEXP, SEXPREC, Rf_allocVector, REAL
 #include "cpp11/as.hpp"         // for as_sexp

inst/include/cpp11/environment.hpp

@@ -1,6 +1,7 @@
 #pragma once
 
-#include <string>             // for string, basic_string
+#include <string>  // for string, basic_string
+
 #include "Rversion.h"         // for R_VERSION, R_Version
 #include "cpp11/R.hpp"        // for SEXP, SEXPREC, Rf_install, Rf_findVarIn...
 #include "cpp11/as.hpp"       // for as_sexp

inst/include/cpp11/external_pointer.hpp

@@ -1,8 +1,9 @@
 #pragma once
 
-#include <cstddef>             // for nullptr_t, NULL
-#include <memory>              // for bad_weak_ptr
-#include <type_traits>         // for add_lvalue_reference
+#include <cstddef>      // for nullptr_t, NULL
+#include <memory>       // for bad_weak_ptr
+#include <type_traits>  // for add_lvalue_reference
+
 #include "cpp11/R.hpp"         // for SEXP, SEXPREC, TYPEOF, R_NilValue, R_C...
 #include "cpp11/protect.hpp"   // for protect, safe, protect::function
 #include "cpp11/r_vector.hpp"  // for type_error