Merge pull request #106 from nextmv-io/develop

Release v0.20.2

Author: merschformann

.github/workflows/go-build-test.yml

@@ -8,6 +8,12 @@ jobs:
       SSH_AUTH_SOCK: /tmp/ssh_agent.sock
       RESOURCES: resources
       NEXTMV_LIBRARY_PATH: ~/.nextmv/lib
+    # Use a matrix strategy to test all the modules simultaneously.
+    strategy:
+      fail-fast: false
+      matrix:
+        MOD_PATH:
+          [./, ./route/google, ./route/here, ./route/osrm, ./route/routingkit]
     steps:
       # Installs dependencies.
       - name: install dependencies linux
@@ -79,6 +85,8 @@ jobs:
 
       - name: go build
         run: go build -v ./...
+        working-directory: ${{ matrix.MOD_PATH }}
 
       - name: go test
         run: NEXTMV_LIBRARY_PATH=${{ env.NEXTMV_LIBRARY_PATH }} NEXTMV_TOKEN=${{ secrets.NEXTMV_TOKEN }} go test ./...
+        working-directory: ${{ matrix.MOD_PATH }}

.github/workflows/go-lint.yml

@@ -3,6 +3,12 @@ on: [push]
 jobs:
   sdk-go-lint:
     runs-on: ubuntu-latest
+    # Use a matrix strategy to test all the modules simultaneously.
+    strategy:
+      fail-fast: false
+      matrix:
+        MOD_PATH:
+          [./, ./route/google, ./route/here, ./route/osrm, ./route/routingkit]
     steps:
       - name: set up go
         uses: actions/setup-go@v3
@@ -17,3 +23,4 @@ jobs:
         uses: golangci/golangci-lint-action@v3
         with:
           version: v1.49.0
+          working-directory: ${{ matrix.MOD_PATH }}

.golangci.yml

@@ -12,7 +12,14 @@ linters-settings:
   gomodguard:
     allowed:
       modules:
-        -
+        - github.com/nextmv-io/sdk
+        - github.com/dgraph-io/ristretto
+        - github.com/golang/mock
+        - github.com/google/go-cmp
+        - github.com/hashicorp/golang-lru
+        - github.com/nextmv-io/go-routingkit
+        - github.com/twpayne/go-polyline
+        - googlemaps.github.io/maps
   # Functions cannot exceed this cyclomatic complexity.
   gocyclo:
     min-complexity: 10
@@ -148,3 +155,24 @@ issues:
     - path: route/example_router_test\.go
       linters:
         - dupl
+
+    # Complexity in UnmarshalJSON is high due to multiple cases, fine for now.
+    - path: route/load\.go
+      linters:
+        - gocyclo
+      text: ByIndexLoader
+
+    # Tag should be 'measure' in both cases ByPoint and ByIndex
+    - path: route/load\.go
+      linters:
+        - tagliatelle
+      text: measure
+
+    # Please note that other Go modules have issues that are ignored but are
+    # not listed here. The reason is that linting must be done standing on the
+    # Go module and excluding issues here uses relative paths. That means that
+    # the relative paths from this file start from the root of sdk but for
+    # other modules the relative path start from that module. For that reason,
+    # linting is ignored by using the following syntax on a line: //nolint. If
+    # you look for uses of //nolint you will find the other linting issues
+    # being excluided.

VERSION

@@ -1 +1 @@
-v0.20.1
+v0.20.2