github/workflow: update to go1.19 (#142)

* github/workflow: update to go1.19

* github/workflows: run gofmt only go1.19

* msgpack,nvim: run gofmt with go1.19

Author: zchee

Diff for: .github/workflows/benchmark.yml

@@ -17,9 +17,10 @@ jobs:
     strategy:
       matrix:
         os:
-          - ubuntu-20.04  # https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-README.md
+          - ubuntu-22.04  # https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md
         go-version:
-          - 1.18.x
+          - 1.19.x
+
     runs-on: ${{ matrix.os }}
 
     steps:
@@ -36,26 +37,6 @@ jobs:
     - name: Checkout code
       uses: actions/checkout@v3
 
-    - name: Cache Go module and build cache
-      uses: actions/cache@v3
-      with:
-        key: ${{ env.OS }}-go-${{ hashFiles('**/go.sum') }}
-        path: |
-          ~/go/pkg/mod              # Module download cache
-          ~/.cache/go-build         # Build cache (Linux)
-        restore-keys: |
-          ${{ env.OS }}-go-
-
-    - name: Cache nvim binary for linux and darwin
-      uses: actions/cache@v3
-      id: cache-nvim
-      with:
-        key: ${{ env.OS }}-nvim-${{ hashFiles('~/nvim/bin/nvim') }}
-        path: |
-          ~/nvim
-        restore-keys: |
-          ${{ env.OS }}-nvim-
-
     - name: Install nvim binary
       uses: rhysd/action-setup-vim@v1
       if: steps.cache-nvim.outputs.cache-hit != 'true'

Diff for: .github/workflows/codeql.yml

@@ -13,7 +13,8 @@ on:
 jobs:
   analyze:
     name: Analyze
-    runs-on: ubuntu-20.04
+    runs-on: 
+      - ubuntu-22.04  # https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md
 
     permissions:
       actions: read

Diff for: .github/workflows/test.yml

@@ -22,13 +22,13 @@ jobs:
     strategy:
       matrix:
         os:
-          - ubuntu-20.04  # https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-README.md
-          - macos-12      # https://github.com/actions/virtual-environments/blob/main/images/macos/macos-12-Readme.md
-          - windows-2022  # https://github.com/actions/virtual-environments/blob/main/images/win/Windows2022-Readme.md
+          - ubuntu-22.04  # https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md
+          - macos-12      # https://github.com/actions/runner-images/blob/main/images/macos/macos-12-Readme.md
+          - windows-2022  # https://github.com/actions/runner-images/blob/main/images/win/Windows2022-Readme.md
         go-version:
-          - 1.16.x
           - 1.17.x
           - 1.18.x
+          - 1.19.x
         neovim-version:
           - v0.7.0
           - nightly
@@ -51,35 +51,26 @@ jobs:
     - name: Checkout code
       uses: actions/checkout@v3
 
-    - name: Cache Go module and build cache
-      uses: actions/cache@v3
-      with:
-        key: go-${{ env.OS }}-${{ hashFiles('**/go.mod') }}
-        path: |
-          ~/go/pkg/mod
-          ~/.cache/go-build
-          ~/Library/Caches/go-build
-          '%LocalAppData%\go-build'
-        restore-keys: |
-          go-${{ env.OS }}-${{ hashFiles('**/go.mod') }}
-
     - name: Install neovim binary
       uses: rhysd/action-setup-vim@v1
       with:
         neovim: true
         version: ${{ matrix.neovim-version }}
 
     - name: gofmt
-      if: ${{ env.OS != 'windows' }}
+      if: matrix.go-version == '1.19.x' && env.OS != 'windows'
       run: |
         diff -u <(echo -n) <(gofmt -s -d .)
 
-    - name: Test and vet
+    - name: go vet
       run: |
         go vet ./...
+
+    - name: Test and take a coverage
+      run: |
         go test -v -race -count=1 -covermode=atomic -coverpkg=./... -coverprofile=coverage.out ./...
 
-    - uses: codecov/codecov-action@v2
+    - uses: codecov/codecov-action@v3
       with:
         file: coverage.out
         flags: ${{ env.OS }}-${{ env.GO_VERSION }}-${{ env.NVIM_VERSION }}

Diff for: msgpack/encode.go

@@ -31,13 +31,13 @@ func encodeUnsupportedType(e *Encoder, v reflect.Value) {
 //
 // Otherwise, Encode uses the following type-dependent default encodings:
 //
-//  Go Type             MessagePack Type
-//  bool                true or false
-//  float32, float64    float64
-//  string              string
-//  []byte              binary
-//  slices, arrays      array
-//  struct, map         map
+//	Go Type             MessagePack Type
+//	bool                true or false
+//	float32, float64    float64
+//	string              string
+//	[]byte              binary
+//	slices, arrays      array
+//	struct, map         map
 //
 // Struct values encode as maps or arrays. If any struct field tag specifies
 // the "array" option, then the struct is encoded as an array. Otherwise, the

Diff for: msgpack/msgpack_test.go

@@ -13,17 +13,17 @@ import (
 
 // pack packs the values vs and returns the result.
 //
-//  Go Type     Encoder method
-//  nil         PackNil
-//  bool        PackBool
-//  int64       PackInt
-//  uint64      PackUint
-//  float64     PackFloat
-//  arrayLen    PackArrayLen
-//  mapLen      PackMapLen
-//  string      PackString(s, false)
-//  []byte      PackBytes(s, true)
-//  extension   PackExtension(k, d)
+//	Go Type     Encoder method
+//	nil         PackNil
+//	bool        PackBool
+//	int64       PackInt
+//	uint64      PackUint
+//	float64     PackFloat
+//	arrayLen    PackArrayLen
+//	mapLen      PackMapLen
+//	string      PackString(s, false)
+//	[]byte      PackBytes(s, true)
+//	extension   PackExtension(k, d)
 func pack(vs ...interface{}) ([]byte, error) {
 	var buf bytes.Buffer
 	enc := NewEncoder(&buf)
@@ -65,17 +65,17 @@ func pack(vs ...interface{}) ([]byte, error) {
 
 // unpack unpacks a byte slice to the following types.
 //
-//   Type      Go
-//   Nil       nil
-//   Bool      bool
-//   Int       int
-//   Uint      int
-//   Float     float64
-//   ArrayLen  arrayLen
-//   MapLen    mapLen
-//   String    string
-//   Binary    []byte
-//   Extension extension
+//	Type      Go
+//	Nil       nil
+//	Bool      bool
+//	Int       int
+//	Uint      int
+//	Float     float64
+//	ArrayLen  arrayLen
+//	MapLen    mapLen
+//	String    string
+//	Binary    []byte
+//	Extension extension
 //
 // This function is not suitable for unpack tests because the integer and float
 // types are mapped to int and float64 respectively.