Skip to content
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.

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

Remove pkg/service and address items flagged by linting #557

Merged
merged 2 commits into from
May 11, 2023

Conversation

sykesm
Copy link
Contributor

@sykesm sykesm commented May 4, 2023

Fixes #548

@thebsdbox
Copy link
Collaborator

oh wow!

@sykesm sykesm marked this pull request as ready for review May 4, 2023 15:23
Copy link
Collaborator

@thebsdbox thebsdbox left a comment

Choose a reason for hiding this comment

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

This is excellent.. :-D

@thebsdbox
Copy link
Collaborator

Apologies @sykesm I merged these in the wrong order, are you ok to fix the merge commits (some of /pkg/service is re-added)

The pkg/service package is no longer used.

Signed-off-by: Matthew Sykes <matthew.sykes@gmail.com>
Signed-off-by: Matthew Sykes <matthew.sykes@gmail.com>
@thebsdbox
Copy link
Collaborator

Awesome, is this ready to go? I'd like to get it merged in before I break things again 😀

@sykesm
Copy link
Contributor Author

sykesm commented May 11, 2023

Awesome, is this ready to go?

Should be. It was pretty straightforward.

@thebsdbox thebsdbox merged commit 2796185 into kube-vip:main May 11, 2023
@sykesm sykesm deleted the kill-pkg-service branch May 11, 2023 19:38
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.

Remove pkg/service
2 participants