Support for .proto files with clang-format (#4198)

.clang-format

@@ -128,4 +128,9 @@ StatementMacros:
 TabWidth:        2
 UseCRLF:         false
 UseTab:          Never
+---
+Language: Proto
+IndentWidth:     2
+UseTab:          Never
+BreakBeforeBraces: Stroustrup
 ...

scripts/check-format.sh

@@ -31,7 +31,7 @@ fi
 file_name_regex="^[[:lower:]0-9_]+$"
 unformatted_files=""
 badly_named_files=""
-for file in $(git ls-files "$@" | grep -e '\.h$' -e '\.hpp$' -e '\.cpp$' -e '\.c$'); do
+for file in $(git ls-files "$@" | grep -e '\.h$' -e '\.hpp$' -e '\.cpp$' -e '\.c$' -e '\.proto$'); do
   if ! clang-format-10 -n -Werror -style=file "$file"; then
     if $fix ; then
       clang-format-10 -style=file -i "$file"

scripts/ci-checks.sh

@@ -46,7 +46,7 @@ else
 fi
 
 echo "$CHECK_DELIMITER"
-echo "-- C/C++ format"
+echo "-- C/C++/Proto format"
 if [ $FIX -ne 0 ]; then
   "$SCRIPT_DIR"/check-format.sh -f include src samples
 else

src/apps/external_executor/protobuf/executor_registration.proto

@@ -16,7 +16,7 @@ service ExecutorRegistration
   // the given client cert.
   // Returns indication of whether the registration succeeded, and error
   // details where appropriate.
-  rpc RegisterExecutor (NewExecutor) returns (RegistrationResult) {}
+  rpc RegisterExecutor(NewExecutor) returns (RegistrationResult) {}
 }
 
 message Attestation

src/apps/external_executor/protobuf/kv.proto

@@ -13,28 +13,28 @@ service KV
 {
   // Indicate to node that this executor is ready for a new request.
   // Returns description of a client request, to be executed.
-  rpc StartTx (google.protobuf.Empty) returns (stream RequestDescription) {}
+  rpc StartTx(google.protobuf.Empty) returns (stream RequestDescription) {}
 
   // Indicate that this executor has completed execution of the previous Tx.
   // Contains description of response which should be forwarded to user.
-  rpc EndTx (ResponseDescription) returns (google.protobuf.Empty) {}
+  rpc EndTx(ResponseDescription) returns (google.protobuf.Empty) {}
 
   // Read a single key
-  rpc Get (KVKey) returns (KVValue) {}
+  rpc Get(KVKey) returns (KVValue) {}
 
   // Test a single key
-  rpc Has (KVKey) returns (KVHasResult) {}
+  rpc Has(KVKey) returns (KVHasResult) {}
 
   // Read version of a single key
-  rpc GetVersion (KVKey) returns (KVVersion) {}
+  rpc GetVersion(KVKey) returns (KVVersion) {}
 
   // Write a single key-value pair
-  rpc Put (KVKeyValue) returns (google.protobuf.Empty) {}
+  rpc Put(KVKeyValue) returns (google.protobuf.Empty) {}
 
   // Remove a single key-value pair
-  rpc Delete (KVKey) returns (google.protobuf.Empty) {}
+  rpc Delete(KVKey) returns (google.protobuf.Empty) {}
 
-  rpc GetAll (KVTable) returns (stream KVKeyValue) {}
+  rpc GetAll(KVTable) returns (stream KVKeyValue) {}
 }
 
 message Header
@@ -51,10 +51,7 @@ message RequestDescription
   bytes body = 4;
 }
 
-message KVTable
-{
-  string table = 1;
-}
+message KVTable { string table = 1; }
 
 message KVKey
 {
@@ -69,20 +66,11 @@ message KVKeyValue
   bytes value = 3;
 }
 
-message KVValue
-{
-  bytes value = 1;
-}
+message KVValue { bytes value = 1; }
 
-message KVHasResult
-{
-  bool present = 1;
-}
+message KVHasResult { bool present = 1; }
 
-message KVVersion
-{
-  uint64 version = 1;
-}
+message KVVersion { uint64 version = 1; }
 
 message ResponseDescription
 {