Skip to content

CHORE: Fix golint warnings about stuttering#2718

Merged
tlimoncelli merged 4 commits intomasterfrom
tlim_dnstutter
Dec 12, 2023
Merged

CHORE: Fix golint warnings about stuttering#2718
tlimoncelli merged 4 commits intomasterfrom
tlim_dnstutter

Conversation

@tlimoncelli
Copy link
Copy Markdown
Contributor

CC @blackshadev

Fix these golint warnings:

pkg/dnsgraph/dnsgraph.go:15:6: type name will be used as dnsgraph.DNSGraphEdge by other packages, and that stutters; consider calling this Edge
pkg/dnsgraph/dnsgraph.go:22:6: type name will be used as dnsgraph.DNSGraphEdges by other packages, and that stutters; consider calling this Edges
pkg/dnsgraph/dnsgraph.go:25:6: type name will be used as dnsgraph.DNSGraphNode by other packages, and that stutters; consider calling this Node

I used VSCode's "rename symbol" which seems to take care of all the right things. However I'm splitting this out as its own PR in case we need to roll it back.

@tlimoncelli
Copy link
Copy Markdown
Contributor Author

@blackshadev I won't merge this until I hear from you. No rush.

@blackshadev
Copy link
Copy Markdown
Collaborator

Go ahead and merge. It is a great suggestion, I love these kind of automated tools the whole reason for the duplicate naming was because it first didn't have it's own package. And when I moved it to it's own package I was busy making things work and then forgot.

@tlimoncelli
Copy link
Copy Markdown
Contributor Author

👍

@tlimoncelli tlimoncelli merged commit 2e4aa7a into master Dec 12, 2023
@tlimoncelli tlimoncelli deleted the tlim_dnstutter branch December 12, 2023 15:36
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