[skip changelog] Update proto files build step

Author: silvanocerza

.github/workflows/link-validation.yaml

@@ -25,11 +25,11 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v2
         with:
-          go-version: "1.14"
+          go-version: "1.16"
 
       - name: Installs Go dependencies
         shell: bash
-        run: go get -u github.com/pseudomuto/protoc-gen-doc/cmd/protoc-gen-doc
+        run: go install github.com/pseudomuto/protoc-gen-doc/cmd/protoc-gen-doc@v1.4.1
 
       - name: Install protoc compiler
         uses: arduino/setup-protoc@v1

.github/workflows/test.yaml

@@ -33,6 +33,7 @@ jobs:
           go get golang.org/x/lint/golint
           go get github.com/golang/protobuf/protoc-gen-go
           go get github.com/cmaglie/go.rice/rice
+          go get google.golang.org/grpc/cmd/protoc-gen-go-grpc
         shell: bash
 
       - name: Install Taskfile

.github/workflows/validate-docs.yaml

@@ -39,12 +39,12 @@ jobs:
       - name: Setup Go
         uses: actions/setup-go@v2
         with:
-          go-version: "1.14"
+          go-version: "1.16"
 
       - name: Install Go dependencies
         run: |
           go version
-          go get -u github.com/pseudomuto/protoc-gen-doc/cmd/protoc-gen-doc
+          go install github.com/pseudomuto/protoc-gen-doc/cmd/protoc-gen-doc@v1.4.1
 
       - name: Install protoc compiler
         uses: arduino/setup-protoc@v1

DistTasks.yml

@@ -20,11 +20,11 @@ tasks:
   all:
     desc: Build for distribution for all platforms
     cmds:
+      - task: Linux_ARMv6
       - task: Windows_32bit
       - task: Windows_64bit
       - task: Linux_32bit
       - task: Linux_64bit
-      - task: Linux_ARMv6
       - task: Linux_ARMv7
       - task: Linux_ARM64
       - task: macOS_64bit
@@ -207,5 +207,5 @@ tasks:
 
 vars:
   CONTAINER: "docker.elastic.co/beats-dev/golang-crossbuild"
-  GO_VERSION: "1.14.7"
+  GO_VERSION: "1.15.10"
   CHECKSUM_FILE: "{{ .VERSION }}-checksums.txt"

Taskfile.yml

@@ -86,10 +86,10 @@ tasks:
   protoc:compile:
     desc: Compile protobuf definitions
     cmds:
-      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=plugins=grpc,paths=source_relative:rpc ./rpc/cc/arduino/cli/commands/v1/*.proto'
-      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=plugins=grpc,paths=source_relative:rpc ./rpc/cc/arduino/cli/monitor/v1/*.proto'
-      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=plugins=grpc,paths=source_relative:rpc ./rpc/cc/arduino/cli/settings/v1/*.proto'
-      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=plugins=grpc,paths=source_relative:rpc ./rpc/cc/arduino/cli/debug/v1/*.proto'
+      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=./rpc --go_opt=paths=source_relative --go-grpc_out=./rpc --go-grpc_opt=paths=source_relative ./rpc/cc/arduino/cli/commands/v1/*.proto'
+      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=./rpc --go_opt=paths=source_relative --go-grpc_out=./rpc --go-grpc_opt=paths=source_relative ./rpc/cc/arduino/cli/monitor/v1/*.proto'
+      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=./rpc --go_opt=paths=source_relative --go-grpc_out=./rpc --go-grpc_opt=paths=source_relative ./rpc/cc/arduino/cli/settings/v1/*.proto'
+      - '{{ default "protoc" .PROTOC_BINARY }} --proto_path=rpc --go_out=./rpc --go_opt=paths=source_relative --go-grpc_out=./rpc --go-grpc_opt=paths=source_relative ./rpc/cc/arduino/cli/debug/v1/*.proto'
 
   protoc:docs:
     desc: Generate docs for protobuf definitions

commands/daemon/daemon.go

@@ -34,6 +34,7 @@ import (
 
 // ArduinoCoreServerImpl FIXMEDOC
 type ArduinoCoreServerImpl struct {
+	rpc.UnimplementedArduinoCoreServiceServer
 	VersionString string
 }
 

commands/daemon/debug.go

@@ -26,7 +26,9 @@ import (
 )
 
 // DebugService implements the `Debug` service
-type DebugService struct{}
+type DebugService struct {
+	dbg.UnimplementedDebugServiceServer
+}
 
 // Debug returns a stream response that can be used to fetch data from the
 // target. The first message passed through the `Debug` request must

commands/daemon/monitor.go

@@ -25,7 +25,9 @@ import (
 )
 
 // MonitorService implements the `Monitor` service
-type MonitorService struct{}
+type MonitorService struct {
+	rpc.UnimplementedMonitorServiceServer
+}
 
 // StreamingOpen returns a stream response that can be used to fetch data from the
 // monitor target. The first message passed through the `StreamingOpenReq` must

commands/daemon/settings.go

@@ -27,7 +27,9 @@ import (
 )
 
 // SettingsService implements the `Settings` service
-type SettingsService struct{}
+type SettingsService struct {
+	rpc.UnimplementedSettingsServiceServer
+}
 
 // GetAll returns a message with a string field containing all the settings
 // currently in use, marshalled in JSON format.