Skip to content
This repository has been archived by the owner on Sep 19, 2018. It is now read-only.

remove default target and ignore functionality #12

Merged
merged 1 commit into from
Nov 9, 2017
Merged

Conversation

alkar
Copy link
Contributor

@alkar alkar commented Nov 8, 2017

  • the default target can be confusing, explicitly setting the target for every Ingress turns out to be a better solution
  • by losing the default target, the ignore label has no reason to exist

Overall, this PR should reduce complexity.

@alkar alkar merged commit b6b6994 into master Nov 9, 2017
@alkar alkar deleted the remove-default branch November 9, 2017 10:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants