Maintenance: use net/netip when upgrade to Go 1.18 is possible #177

Closed
opened 2025-11-20 04:21:13 -05:00 by saavagebueno · 3 comments
Owner

Originally created by @qdm12 on GitHub (Mar 9, 2022).

Originally assigned to: @qdm12 on GitHub.

  1. What's the feature?

Use https://pkg.go.dev/net/netip which is more robust and better than the current net.IP addresses

  1. Extra information?

This needs Go 1.18 which doesn't work with golangci-lint currently, so we'll have to wait.

Originally created by @qdm12 on GitHub (Mar 9, 2022). Originally assigned to: @qdm12 on GitHub. 1. What's the feature? Use https://pkg.go.dev/net/netip which is more robust and better than the current `net.IP` addresses 3. Extra information? This needs Go 1.18 which doesn't work with golangci-lint currently, so we'll have to wait.
Author
Owner

@rgm3 commented on GitHub (Apr 30, 2022):

golangci-lint supports go 1.18 since v1.45.0, although some linters that do not support go 1.18 may be disabled. For details, see https://github.com/golangci/golangci-lint/issues/2649.

@rgm3 commented on GitHub (Apr 30, 2022): golangci-lint supports go 1.18 since v1.45.0, although some linters that do not support go 1.18 may be disabled. For details, see https://github.com/golangci/golangci-lint/issues/2649.
Author
Owner

@qdm12 commented on GitHub (Apr 30, 2022):

Yes that's rather unfortunate a bunch of linters are not compatible. Thanks for mentioning this.
Since this issue is not really urgent, I'lll probably delay the upgrade to Go 1.18 until all golangci-lint linters are working again.

@qdm12 commented on GitHub (Apr 30, 2022): Yes that's rather unfortunate a bunch of linters are not compatible. Thanks for mentioning this. Since this issue is not really urgent, I'lll probably delay the upgrade to Go 1.18 until all golangci-lint linters are working again.
Author
Owner

@qdm12 commented on GitHub (Jun 12, 2023):

Done in 9a4a268926

@qdm12 commented on GitHub (Jun 12, 2023): Done in 9a4a268926c73f360578cf4a2a9d75a156541250
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ddns-updater#177