-
Notifications
You must be signed in to change notification settings - Fork 500
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
Add plugin list page #202
Add plugin list page #202
Conversation
Introduces a lightweight plugins.md that can be linked in onboarding docs at the "krew" repo as well as this "krew-index" repo. Supersedes kubernetes-sigs/krew#303. Signed-off-by: Ahmet Alp Balkan <[email protected]>
I was thinking about how to include this in krew-index but avoid distributing to our users. One idea would be to use On second thought: maybe an easier approach is to have a dedicated branch where the plugins.md lives. I'm quite sure this does work :) |
The file is really small, like a plugin manifest. So I'm not too worried about it. It's like having a plugin that updates occassionally. |
One final thought before merging: as this file is not automatically updated, I think we should include a disclaimer on top that this may be outdated -- and say what revision this file refers to. WDYT? |
Yeah that's possible, we should say run |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahmetb, corneliusweig The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Introduces a lightweight plugins.md that can be linked in onboarding docs
at the "krew" repo as well as this "krew-index" repo.
Supersedes kubernetes-sigs/krew#303.