Skip to content

golangci-lint: use gopacket/gopacket instead of google/gopacket #40321

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 2, 2025
Merged

golangci-lint: use gopacket/gopacket instead of google/gopacket #40321

merged 1 commit into from
Jul 2, 2025

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Jul 2, 2025

Commit 7a85cf1 changed the eintire code base to use the github.com/gopacket/gopacket package instead of
github.com/google/gopacket. Add a lint check to avoid accidentally re-adding the dependency.

For #34536

@tklauser
golangci-lint: use gopacket/gopacket instead of google/gopacket
66d6277 
Commit 7a85cf1 changed the eintire code base to use the
github.com/gopacket/gopacket package instead of
github.com/google/gopacket. Add a lint check to avoid accidentally
re-adding the dependency.

For #34536

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser requested a review from a team as a code owner July 2, 2025 08:32
@tklauser tklauser requested a review from christarazi July 2, 2025 08:32
@tklauser tklauser added the release-note/ci This PR makes changes to the CI. label Jul 2, 2025
@tklauser
Copy link
Member Author

tklauser commented Jul 2, 2025

/test

@tklauser tklauser added dont-merge/blocked Another PR must be merged before this one. and removed dont-merge/blocked Another PR must be merged before this one. labels Jul 2, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 2, 2025
@tklauser tklauser added this pull request to the merge queue Jul 2, 2025
Merged via the queue into main with commit de6ad15 Jul 2, 2025
329 of 334 checks passed
@tklauser tklauser deleted the pr/tklauser/lint-gopacket branch July 2, 2025 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@christarazi christarazi christarazi approved these changes

Assignees
No one assigned
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tklauser @christarazi