Skip to content

Replace retired Go Report Card badge - #181

Merged
xylo04 merged 4 commits into
mainfrom
copilot/replace-go-report-card-badge
Sep 8, 2026
Merged

Replace retired Go Report Card badge#181
xylo04 merged 4 commits into
mainfrom
copilot/replace-go-report-card-badge

Conversation

Copilot AI commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The Go Report Card service has been retired and its README badge is no longer useful.

  • Replace the Go Report Card badge with the official golangci-lint badge.
  • Link the badge to the repository’s golangci-lint results page.
[![golangci-lint](https://golangci-lint.run/badges/github.com/k0swe/kel-agent.svg)](https://golangci-lint.run/github.com/k0swe/kel-agent)

Copilot AI linked an issue Sep 8, 2026 that may be closed by this pull request
Co-authored-by: xylo04 <389971+xylo04@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Go Report Card badge to GolangCI-Lint Replace retired Go Report Card badge Sep 8, 2026
Copilot AI requested a review from xylo04 September 8, 2026 14:28
@xylo04
xylo04 marked this pull request as ready for review September 8, 2026 14:36
Co-authored-by: xylo04 <389971+xylo04@users.noreply.github.com>
@xylo04

xylo04 commented Sep 8, 2026

Copy link
Copy Markdown
Member

Co-authored-by: xylo04 <389971+xylo04@users.noreply.github.com>

Copilot AI commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Add a CI integration per https://golangci-lint.run/docs/welcome/install/ci/#github-actions.

Added a dedicated golangci-lint GitHub Actions workflow using the official golangci/golangci-lint-action@v9, pinned to linter v2.13, in commit ee5bcef.

@xylo04
xylo04 merged commit ed91d3c into main Sep 8, 2026
3 checks passed
@xylo04
xylo04 deleted the copilot/replace-go-report-card-badge branch September 8, 2026 14:50
xylo04 added a commit to k0swe/rigctld-go that referenced this pull request Sep 8, 2026
* Replace retired Go Report Card badge

The Go Report Card service has been retired and its README badge is no
longer useful. Remove the badge and add a golangci-lint workflow.

Replicates k0swe/kel-agent#181.

* Use fmt.Errorf instead of errors.New(fmt.Sprintf) (staticcheck S1028)

---------

Co-authored-by: xylo04 <chris@xylo.net>
xylo04 added a commit to k0swe/qrz-logbook that referenced this pull request Sep 8, 2026
* Replace retired Go Report Card badge

The Go Report Card service has been retired and its README badge is no
longer useful. Remove the badge and add a golangci-lint workflow.

Replicates k0swe/kel-agent#181.

* Use strings.ReplaceAll (staticcheck QF1004)

---------

Co-authored-by: xylo04 <chris@xylo.net>
xylo04 added a commit to k0swe/qrz-api that referenced this pull request Sep 8, 2026
The Go Report Card service has been retired and its README badge is no
longer useful. Remove the badge and add a golangci-lint workflow.

Replicates k0swe/kel-agent#181.

Co-authored-by: xylo04 <chris@xylo.net>
xylo04 added a commit to k0swe/lotw-qsl that referenced this pull request Sep 8, 2026
The Go Report Card service has been retired and its README badge is no
longer useful. Remove the badge and add a golangci-lint workflow.

Replicates k0swe/kel-agent#181.

Co-authored-by: xylo04 <chris@xylo.net>
xylo04 added a commit to k0swe/wsjtx-go that referenced this pull request Sep 8, 2026
* Replace retired Go Report Card badge

The Go Report Card service has been retired and its README badge is no
longer useful. Remove the badge and add a golangci-lint workflow.

Replicates k0swe/kel-agent#181.

* Target v4 default branch in golangci-lint triggers

* Fix golangci-lint findings

- parser.go: replace ineffectual 'err' assignments with '_' where the
  error was already being discarded (ineffassign). Error propagation is
  unchanged: the final assignment before each return still uses err.
- Suppress ST1012 on exported error vars ParseError / NotConnectedError
  with //nolint and a reason: renaming them would break the published
  v4 module's public API.

All existing tests pass; no behavior change.

---------

Co-authored-by: xylo04 <chris@xylo.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace Go Report Card

2 participants