diff --git a/.github/workflows/bench.yml b/.github/workflows/bench.yml
index bde132f..d750b9d 100644
--- a/.github/workflows/bench.yml
+++ b/.github/workflows/bench.yml
@@ -12,21 +12,34 @@ on:
description: 'New Ref'
required: true
+# Cancel the workflow in progress in newer build is about to start.
+concurrency:
+ group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+ cancel-in-progress: true
+
env:
GO111MODULE: "on"
CACHE_BENCHMARK: "off" # Enables benchmark result reuse between runs, may skew latency results.
RUN_BASE_BENCHMARK: "on" # Runs benchmark for PR base in case benchmark result is missing.
+ GO_VERSION: 1.19.x
jobs:
bench:
- strategy:
- matrix:
- go-version: [ 1.16.x ]
runs-on: ubuntu-latest
steps:
- - name: Install Go
- uses: actions/setup-go@v2
+ - name: Install Go stable
+ if: env.GO_VERSION != 'tip'
+ uses: actions/setup-go@v3
with:
- go-version: ${{ matrix.go-version }}
+ go-version: ${{ env.GO_VERSION }}
+ - name: Install Go tip
+ if: env.GO_VERSION == 'tip'
+ run: |
+ curl -sL https://storage.googleapis.com/go-build-snap/go/linux-amd64/$(git ls-remote https://github.com/golang/go.git HEAD | awk '{print $1;}').tar.gz -o gotip.tar.gz
+ ls -lah gotip.tar.gz
+ mkdir -p ~/sdk/gotip
+ tar -C ~/sdk/gotip -xzf gotip.tar.gz
+ ~/sdk/gotip/bin/go version
+ echo "PATH=$HOME/go/bin:$HOME/sdk/gotip/bin/:$PATH" >> $GITHUB_ENV
- name: Checkout code
uses: actions/checkout@v2
with:
@@ -74,13 +87,17 @@ jobs:
id: bench
run: |
export REF_NAME=new
- BENCH_COUNT=5 make bench-run bench-stat
+ BENCH_COUNT=5 make bench
+ OUTPUT=$(make bench-stat-diff)
+ echo "${OUTPUT}"
+ OUTPUT="${OUTPUT//$'\n'/%0A}"
+ echo "::set-output name=diff::$OUTPUT"
OUTPUT=$(make bench-stat)
- OUTPUT="${OUTPUT//'%'/'%25'}"
- OUTPUT="${OUTPUT//$'\n'/'%0A'}"
- OUTPUT="${OUTPUT//$'\r'/'%0D'}"
+ echo "${OUTPUT}"
+ OUTPUT="${OUTPUT//$'\n'/%0A}"
echo "::set-output name=result::$OUTPUT"
- name: Comment Benchmark Result
+ continue-on-error: true
uses: marocchino/sticky-pull-request-comment@v2
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -89,6 +106,13 @@ jobs:
### Benchmark Result
Benchmark diff with base branch
+ ```
+ ${{ steps.bench.outputs.diff }}
+ ```
+ Benchmark result
+
```
${{ steps.bench.outputs.result }}
```
diff --git a/.github/workflows/cloc.yml b/.github/workflows/cloc.yml
index ed1ddb9..4592bb4 100644
--- a/.github/workflows/cloc.yml
+++ b/.github/workflows/cloc.yml
@@ -2,6 +2,12 @@
name: cloc
on:
pull_request:
+
+# Cancel the workflow in progress in newer build is about to start.
+concurrency:
+ group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+ cancel-in-progress: true
+
jobs:
cloc:
runs-on: ubuntu-latest
@@ -18,14 +24,14 @@ jobs:
- name: Count Lines Of Code
id: loc
run: |
- curl -OL https://github.com/vearutop/sccdiff/releases/download/v1.0.1/linux_amd64.tar.gz && tar xf linux_amd64.tar.gz
+ curl -sLO https://github.com/vearutop/sccdiff/releases/download/v1.0.2/linux_amd64.tar.gz && tar xf linux_amd64.tar.gz && echo "b17e76bede22af0206b4918d3b3c4e7357f2a21b57f8de9e7c9dc0eb56b676c0 sccdiff" | shasum -c
OUTPUT=$(cd pr && ../sccdiff -basedir ../base)
- OUTPUT="${OUTPUT//'%'/'%25'}"
- OUTPUT="${OUTPUT//$'\n'/'%0A'}"
- OUTPUT="${OUTPUT//$'\r'/'%0D'}"
+ echo "${OUTPUT}"
+ OUTPUT="${OUTPUT//$'\n'/%0A}"
echo "::set-output name=diff::$OUTPUT"
- name: Comment Code Lines
+ continue-on-error: true
uses: marocchino/sticky-pull-request-comment@v2
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
diff --git a/.github/workflows/golangci-lint.yml b/.github/workflows/golangci-lint.yml
index a7977d6..54723e6 100644
--- a/.github/workflows/golangci-lint.yml
+++ b/.github/workflows/golangci-lint.yml
@@ -8,17 +8,26 @@ on:
- master
- main
pull_request:
+
+# Cancel the workflow in progress in newer build is about to start.
+concurrency:
+ group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+ cancel-in-progress: true
+
jobs:
golangci:
name: golangci-lint
runs-on: ubuntu-latest
steps:
+ - uses: actions/setup-go@v3
+ with:
+ go-version: 1.19.x
- uses: actions/checkout@v2
- name: golangci-lint
- uses: golangci/golangci-lint-action@v2.5.2
+ uses: golangci/golangci-lint-action@v3.1.0
with:
# Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
- version: v1.40.1
+ version: v1.48.0
# Optional: working directory, useful for monorepos
# working-directory: somedir
diff --git a/.github/workflows/gorelease.yml b/.github/workflows/gorelease.yml
new file mode 100644
index 0000000..6267500
--- /dev/null
+++ b/.github/workflows/gorelease.yml
@@ -0,0 +1,57 @@
+# This script is provided by github.com/bool64/dev.
+name: gorelease
+on:
+ pull_request:
+
+# Cancel the workflow in progress in newer build is about to start.
+concurrency:
+ group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+ cancel-in-progress: true
+
+env:
+ GO_VERSION: 1.19.x
+jobs:
+ gorelease:
+ runs-on: ubuntu-latest
+ steps:
+ - name: Install Go stable
+ if: env.GO_VERSION != 'tip'
+ uses: actions/setup-go@v3
+ with:
+ go-version: ${{ env.GO_VERSION }}
+ - name: Install Go tip
+ if: env.GO_VERSION == 'tip'
+ run: |
+ curl -sL https://storage.googleapis.com/go-build-snap/go/linux-amd64/$(git ls-remote https://github.com/golang/go.git HEAD | awk '{print $1;}').tar.gz -o gotip.tar.gz
+ ls -lah gotip.tar.gz
+ mkdir -p ~/sdk/gotip
+ tar -C ~/sdk/gotip -xzf gotip.tar.gz
+ ~/sdk/gotip/bin/go version
+ echo "PATH=$HOME/go/bin:$HOME/sdk/gotip/bin/:$PATH" >> $GITHUB_ENV
+ - name: Checkout code
+ uses: actions/checkout@v2
+ - name: Gorelease cache
+ uses: actions/cache@v2
+ with:
+ path: |
+ ~/go/bin/gorelease
+ key: ${{ runner.os }}-gorelease-generic
+ - name: Gorelease
+ id: gorelease
+ run: |
+ test -e ~/go/bin/gorelease || go install golang.org/x/exp/cmd/gorelease@latest
+ OUTPUT=$(gorelease 2>&1 || exit 0)
+ echo "${OUTPUT}"
+ OUTPUT="${OUTPUT//$'\n'/%0A}"
+ echo "::set-output name=report::$OUTPUT"
+ - name: Comment Report
+ continue-on-error: true
+ uses: marocchino/sticky-pull-request-comment@v2
+ with:
+ header: gorelease
+ message: |
+ ### Go API Changes
+
+
+ ${{ steps.gorelease.outputs.report }}
+
\ No newline at end of file
diff --git a/.github/workflows/test-unit.yml b/.github/workflows/test-unit.yml
index 7304bfc..eeacf67 100644
--- a/.github/workflows/test-unit.yml
+++ b/.github/workflows/test-unit.yml
@@ -6,22 +6,43 @@ on:
- master
- main
pull_request:
+
+# Cancel the workflow in progress in newer build is about to start.
+concurrency:
+ group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+ cancel-in-progress: true
+
env:
GO111MODULE: "on"
RUN_BASE_COVERAGE: "on" # Runs test for PR base in case base test coverage is missing.
+ COV_GO_VERSION: 1.18.x # Version of Go to collect coverage
+ TARGET_DELTA_COV: 90 # Target coverage of changed lines, in percents
jobs:
test:
strategy:
matrix:
- go-version: [ 1.13.x, 1.14.x, 1.15.x, 1.16.x ]
+ go-version: [ 1.18.x, 1.19.x ]
runs-on: ubuntu-latest
steps:
- - name: Install Go
- uses: actions/setup-go@v2
+ - name: Install Go stable
+ if: matrix.go-version != 'tip'
+ uses: actions/setup-go@v3
with:
go-version: ${{ matrix.go-version }}
+
+ - name: Install Go tip
+ if: matrix.go-version == 'tip'
+ run: |
+ curl -sL https://storage.googleapis.com/go-build-snap/go/linux-amd64/$(git ls-remote https://github.com/golang/go.git HEAD | awk '{print $1;}').tar.gz -o gotip.tar.gz
+ ls -lah gotip.tar.gz
+ mkdir -p ~/sdk/gotip
+ tar -C ~/sdk/gotip -xzf gotip.tar.gz
+ ~/sdk/gotip/bin/go version
+ echo "PATH=$HOME/go/bin:$HOME/sdk/gotip/bin/:$PATH" >> $GITHUB_ENV
+
- name: Checkout code
uses: actions/checkout@v2
+
- name: Go cache
uses: actions/cache@v2
with:
@@ -34,42 +55,57 @@ jobs:
key: ${{ runner.os }}-go-cache-${{ hashFiles('**/go.sum') }}
restore-keys: |
${{ runner.os }}-go-cache
+
- name: Restore base test coverage
- if: matrix.go-version == '1.16.x'
+ id: base-coverage
+ if: matrix.go-version == env.COV_GO_VERSION && github.event.pull_request.base.sha != ''
uses: actions/cache@v2
with:
path: |
unit-base.txt
# Use base sha for PR or new commit hash for master/main push in test result key.
key: ${{ runner.os }}-unit-test-coverage-${{ (github.event.pull_request.base.sha != github.event.after) && github.event.pull_request.base.sha || github.event.after }}
- - name: Checkout base code
- if: matrix.go-version == '1.16.x' && env.RUN_BASE_COVERAGE == 'on' && steps.benchmark-base.outputs.cache-hit != 'true' && github.event.pull_request.base.sha != ''
- uses: actions/checkout@v2
- with:
- ref: ${{ github.event.pull_request.base.sha }}
- path: __base
+
- name: Run test for base code
- if: matrix.go-version == '1.16.x' && env.RUN_BASE_COVERAGE == 'on' && steps.benchmark-base.outputs.cache-hit != 'true' && github.event.pull_request.base.sha != ''
+ if: matrix.go-version == env.COV_GO_VERSION && env.RUN_BASE_COVERAGE == 'on' && steps.base-coverage.outputs.cache-hit != 'true' && github.event.pull_request.base.sha != ''
run: |
- cd __base
- make | grep test-unit && (make test-unit && go tool cover -func=./unit.coverprofile | sed -e 's/.go:[0-9]*:\t/.go\t/g' | sed -e 's/\t\t*/\t/g' > ../unit-base.txt) || echo "No test-unit in base"
+ git fetch origin master ${{ github.event.pull_request.base.sha }}
+ HEAD=$(git rev-parse HEAD)
+ git reset --hard ${{ github.event.pull_request.base.sha }}
+ (make test-unit && go tool cover -func=./unit.coverprofile > unit-base.txt) || echo "No test-unit in base"
+ git reset --hard $HEAD
+
- name: Test
id: test
run: |
make test-unit
- go tool cover -func=./unit.coverprofile | sed -e 's/.go:[0-9]*:\t/.go\t/g' | sed -e 's/\t\t*/\t/g' > unit.txt
- OUTPUT=$(test -e unit-base.txt && (diff unit-base.txt unit.txt || exit 0) || cat unit.txt)
- OUTPUT="${OUTPUT//'%'/'%25'}"
- OUTPUT="${OUTPUT//$'\n'/'%0A'}"
- OUTPUT="${OUTPUT//$'\r'/'%0D'}"
+ go tool cover -func=./unit.coverprofile > unit.txt
TOTAL=$(grep 'total:' unit.txt)
- echo "::set-output name=diff::$OUTPUT"
+ echo "${TOTAL}"
echo "::set-output name=total::$TOTAL"
- - name: Store base coverage
- if: ${{ github.ref == 'refs/heads/master' || github.ref == 'refs/heads/main' }}
- run: cp unit.txt unit-base.txt
+
+ - name: Annotate missing test coverage
+ id: annotate
+ if: matrix.go-version == env.COV_GO_VERSION && github.event.pull_request.base.sha != ''
+ run: |
+ curl -sLO https://github.com/vearutop/gocovdiff/releases/download/v1.3.6/linux_amd64.tar.gz && tar xf linux_amd64.tar.gz
+ gocovdiff_hash=$(git hash-object ./gocovdiff)
+ [ "$gocovdiff_hash" == "8e507e0d671d4d6dfb3612309b72b163492f28eb" ] || (echo "::error::unexpected hash for gocovdiff, possible tampering: $gocovdiff_hash" && exit 1)
+ git fetch origin master ${{ github.event.pull_request.base.sha }}
+ REP=$(./gocovdiff -cov unit.coverprofile -gha-annotations gha-unit.txt -delta-cov-file delta-cov-unit.txt -target-delta-cov ${TARGET_DELTA_COV})
+ echo "${REP}"
+ REP="${REP//$'\n'/%0A}"
+ cat gha-unit.txt
+ DIFF=$(test -e unit-base.txt && ./gocovdiff -func-cov unit.txt -func-base-cov unit-base.txt || echo "Missing base coverage file")
+ DIFF="${DIFF//$'\n'/%0A}"
+ TOTAL=$(cat delta-cov-unit.txt)
+ echo "::set-output name=rep::$REP"
+ echo "::set-output name=diff::$DIFF"
+ echo "::set-output name=total::$TOTAL"
+
- name: Comment Test Coverage
- if: matrix.go-version == '1.16.x'
+ continue-on-error: true
+ if: matrix.go-version == env.COV_GO_VERSION && github.event.pull_request.base.sha != ''
uses: marocchino/sticky-pull-request-comment@v2
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -77,15 +113,25 @@ jobs:
message: |
### Unit Test Coverage
${{ steps.test.outputs.total }}
+ ${{ steps.annotate.outputs.total }}
+ Coverage of changed lines
+
+ ${{ steps.annotate.outputs.rep }}
+
+ Coverage diff with base branch
- ```diff
- ${{ steps.test.outputs.diff }}
- ```
+ ${{ steps.annotate.outputs.diff }}
+