Skip to content

bump golanglint-ci.#105

Merged
jrick merged 1 commit intodecred:masterfrom
amass01:golanglint-ci
Jul 18, 2025
Merged

bump golanglint-ci.#105
jrick merged 1 commit intodecred:masterfrom
amass01:golanglint-ci

Conversation

@amass01
Copy link
Member

@amass01 amass01 commented Jul 17, 2025

No description provided.

@amass01 amass01 force-pushed the golanglint-ci branch 2 times, most recently from 81103fb to 357692d Compare July 17, 2025 17:19
Comment on lines -10 to -11
- gofmt
- goimports
Copy link
Member Author

@amass01 amass01 Jul 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added as formatters

- bodyclose
- gofmt
- goimports
- gosimple
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's part of staticcheck

- rowserrcheck
- sqlclosecheck
- tparallel
- typecheck
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's part of govet

@amass01 amass01 force-pushed the golanglint-ci branch 7 times, most recently from 3ed6bc0 to 89fe739 Compare July 17, 2025 20:36
Copy link
Member

@davecgh davecgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've been holding off on merge the PR in dcrd that does this because the last time I checked the release version of gopls doesn't work properly with v2 yet (but it does on the development branch).

If the active developers on this repo don't care about not seeing proper lint warnings in their editor until a new version of gopls with those updates is released, I have no objection to merging it for this repo.

@amass01 amass01 requested a review from jrick July 18, 2025 17:08
@jrick jrick merged commit 0418c83 into decred:master Jul 18, 2025
2 checks passed
@amass01 amass01 deleted the golanglint-ci branch July 18, 2025 18:45
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.

3 participants