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

labels: Move label filtering into pkg/labelsfilter #10894

Closed
wants to merge 1 commit into from

Conversation

tgraf
Copy link
Member

@tgraf tgraf commented Apr 8, 2020

The labels filter has non-trivial dependencies and is not directly
related to the base Labels type.

@tgraf tgraf added the release-note/misc This PR makes changes that have no direct user impact. label Apr 8, 2020
@tgraf tgraf requested a review from a team April 8, 2020 13:49
@tgraf tgraf requested review from a team as code owners April 8, 2020 13:49
@tgraf tgraf requested review from a team April 8, 2020 13:49
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Apr 8, 2020
The labels filter has non-trivial dependencies and is not directly
related to the base Labels type.

Signed-off-by: Thomas Graf <thomas@cilium.io>
@tgraf tgraf force-pushed the pr/tgraf/split-labels-pkg branch from bbe1d9c to e3af3d4 Compare April 8, 2020 16:39
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 46.957% when pulling e3af3d4 on pr/tgraf/split-labels-pkg into 450c79c on master.

@tgraf
Copy link
Member Author

tgraf commented Apr 8, 2020

test-me-please

@tgraf
Copy link
Member Author

tgraf commented Apr 9, 2020

Closing in favor of #10909

@tgraf tgraf closed this Apr 9, 2020
@tgraf tgraf deleted the pr/tgraf/split-labels-pkg branch April 9, 2020 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.8.0
  
In progress
Development

Successfully merging this pull request may close these issues.

None yet

3 participants