Skip to content

Update golangci-lint#488

Merged
aarongable merged 3 commits into
mainfrom
mattm-golangcilint
Feb 25, 2025
Merged

Update golangci-lint#488
aarongable merged 3 commits into
mainfrom
mattm-golangcilint

Conversation

@mcpherrinm

@mcpherrinm mcpherrinm commented Feb 25, 2025

Copy link
Copy Markdown
Contributor

golangci-lint is failing on main due to a config problem. Update configs, version of golangci-lint, and remove a variable that's unused and causing a lint failure

check-shadowing flag to govet is deprecated and causing failures on main

tenv is deprecated, replaced by usetesting
@mcpherrinm

mcpherrinm commented Feb 25, 2025

Copy link
Copy Markdown
Contributor Author
Error: core/types.go:263:61: unused-parameter: parameter 'now' seems to be unused, consider removing or renaming it as _ (revive)
 func RenewalInfoSimple(issued time.Time, expires time.Time, now time.Time) *RenewalInfo {

Seem like a legitimate problem, fixing in another commit

@aarongable aarongable merged commit 307a947 into main Feb 25, 2025
@aarongable aarongable deleted the mattm-golangcilint branch February 25, 2025 16:44
kwatson added a commit to kwatson/letsencrypt-pebble that referenced this pull request Jun 9, 2025
* 'main' of https://github.com/letsencrypt/pebble: (35 commits)
  add overriding of ARI response (letsencrypt#501)
  wfe: fix a race in `orderForDisplay` (letsencrypt#500)
  Bump golang.org/x/ dependencies (letsencrypt#499)
  currectly triggers BadSignatureAlgorithmProblem at JWS parse time (letsencrypt#492)
  use newer validation subdomain for dns-account-01 (fix CI eggsampler/acme error) (letsencrypt#498)
  Orders don't have a "deactivated" status. (letsencrypt#301)
  Update golangci-lint (letsencrypt#488)
  build(deps): bump github.com/go-jose/go-jose/v4 from 4.0.4 to 4.0.5 (letsencrypt#487)
  Truncate ARI timestamps to millisecond resolution (letsencrypt#485)
  return logical and compliant ARI windows for expiring certs (letsencrypt#484)
  Update dependencies (letsencrypt#481)
  docs: rm mention of subproblems being unimpl'd (letsencrypt#479)
  Fix(NOISSUE): Fix docker compose file example in README.md (letsencrypt#475)
  Add support for ACME Profiles (letsencrypt#473)
  Simplify KU, EKU, and SKID fields of issued certs (letsencrypt#472)
  Update golangci-lint to 1.60.2 (letsencrypt#474)
  Update /x/net for compatibility with go1.23 (letsencrypt#470)
  Reject extra command line args and fix README invocation (letsencrypt#467)
  Document exposing API and management ports when not using docker-compose.yaml (letsencrypt#465)
  Implement latest draft-ietf-acme-ari spec (letsencrypt#461)
  ...
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.

2 participants