-
Notifications
You must be signed in to change notification settings - Fork 41
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
read from stdin #125
base: main
Are you sure you want to change the base?
read from stdin #125
Conversation
The committers listed above are authorized under a signed CLA. |
Welcome @duncan485! |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: duncan485 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Just wanted to add that I have a similar usage in mind and I've tested this change locally and verified it behaves as desired. |
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
Looking forward to seeing the feature in production! |
The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
/remove-lifecycle rotten |
What type of PR is this?
feature
What this PR does / why we need it:
Allow reading from stdin , using "-" (just like
kubectl apply -f -
). This also providers the ability to use templating engines and directly pipe their input to kubectl-validate. For examplekustomize build overlays/dev | kubectl validate -
.Which issue(s) this PR fixes:
Fixes #32
Special notes for your reviewer:
First contribution, please let me know if I'm in the right direction with this solution. I know it at least needs some documentation, but wanted to get some feedback first.
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: