Skip to content

Commit c13920b

Browse files
committed
ci(workflow): simplify GitHub Actions workflow by removing separate coverage job
docs(README.md): add reference to codeGROOVE-dev/retry project in SEE ALSO section docs(retry.go): update SEE ALSO section with new project reference
1 parent ddfd5fb commit c13920b

File tree

3 files changed

+15
-16
lines changed

3 files changed

+15
-16
lines changed

.github/workflows/workflow.yaml

Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -29,36 +29,28 @@ jobs:
2929
GOVERSION: ${{ matrix.go-version }}
3030
steps:
3131
- uses: actions/checkout@v4
32+
3233
- name: Setup Go ${{ matrix.go-version }}
3334
uses: actions/setup-go@v5
3435
with:
3536
go-version: ${{ matrix.go-version }}
3637
check-latest: true
3738
cache: true
39+
3840
- name: Display Go version
3941
run: go version
42+
4043
- name: Install dependencies
4144
run: make setup
45+
4246
- name: Test
4347
run: make ci
44-
- name: Archive code coverage results
45-
uses: actions/upload-artifact@v4
46-
with:
47-
name: code-coverage-report-${{ matrix.os }}-${{ matrix.go-version }}
48-
path: coverage.txt
4948

50-
coverage:
51-
needs: tests
52-
runs-on: ubuntu-latest
53-
54-
steps:
55-
- name: Download a linux coverage report
56-
uses: actions/download-artifact@v4
57-
with:
58-
name: code-coverage-report-ubuntu-latest-1.24
5949
- name: Upload coverage to Codecov
50+
if: matrix.os == 'ubuntu-latest' && matrix.go-version == '1.25'
6051
uses: codecov/codecov-action@v4
6152
with:
6253
token: ${{ secrets.CODECOV_TOKEN }}
63-
fail_ci_if_error: true
54+
files: ./coverage.txt
6455
flags: unittest
56+
fail_ci_if_error: true

README.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,10 @@ HTTP GET with retry with data:
7171

7272
# SEE ALSO
7373

74+
* [codeGROOVE-dev/retry](https://github.com/codeGROOVE-dev/retry) - Modern fork
75+
of avast/retry-go/v4 focused on correctness, reliability and efficiency. 100%
76+
API-compatible drop-in replacement. Looks really good.
77+
7478
* [giantswarm/retry-go](https://github.com/giantswarm/retry-go) - slightly
7579
complicated interface.
7680

@@ -317,7 +321,8 @@ Delay set delay between retry default is 100ms
317321
```go
318322
func DelayType(delayType DelayTypeFunc) Option
319323
```
320-
DelayType set type of the delay between retries default is BackOff
324+
DelayType set type of the delay between retries default is a combination of
325+
BackOffDelay and RandomDelay for exponential backoff with jitter
321326
322327
#### func LastErrorOnly
323328

retry.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,8 @@ HTTP GET with retry with data:
6161
6262
# SEE ALSO
6363
64+
* [codeGROOVE-dev/retry](https://github.com/codeGROOVE-dev/retry) - Modern fork of avast/retry-go/v4 focused on correctness, reliability and efficiency. 100% API-compatible drop-in replacement. Looks really good.
65+
6466
* [giantswarm/retry-go](https://github.com/giantswarm/retry-go) - slightly complicated interface.
6567
6668
* [sethgrid/pester](https://github.com/sethgrid/pester) - only http retry for http calls with retries and backoff

0 commit comments

Comments
 (0)