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

Add key action for + to do kubectl apply -f #1443

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yeahdongcn
Copy link
Contributor

k9s is such a great tool for my daily work. The only problem is I have to switch to a different terminal tab or window if I want to apply new YAML files.
This PR added key action for + to do kubectl apply -f. The process can be described as follows.

  1. Copy YAML file content from your browser or somewhere else
  2. Press + (shift + =) to open a temp file
  3. Paste it then save and quit
  4. kubectl apply -f

@yeahdongcn
Copy link
Contributor Author

Maybe a cure for #1323?

@derailed
Copy link
Owner

derailed commented Apr 3, 2022

@yeahdongcn Thank you for this PR. Do you know the dir command provides for listing and applying manifests?

@derailed derailed added the question Further information is requested label Apr 3, 2022
@yeahdongcn
Copy link
Contributor Author

Oh, I didn't realize there was a similar command already implemented. dir looks awesome! But this PR actually tries to fulfill another user scenario that one can use an online manifest w/o creating a local file.

@derailed
Copy link
Owner

@yeahdongcn Thank you for the clarification! Good point. Looks like we have some conflicts. Could you update?

@derailed derailed added enhancement New feature or request needs-tlc Pr needs additional updates and removed question Further information is requested labels Nov 12, 2023
@yeahdongcn yeahdongcn closed this Nov 13, 2023
@yeahdongcn yeahdongcn reopened this Nov 13, 2023
@yeahdongcn
Copy link
Contributor Author

yeahdongcn commented Nov 13, 2023

@yeahdongcn Thank you for the clarification! Good point. Looks like we have some conflicts. Could you update?

@derailed Done. Please see the latest commit. Thanks.

@ryaminal
Copy link

learned about :dir . from this PR, super helpful

also, this PR is super exciting to see for such an amazing tool.

@yeahdongcn
Copy link
Contributor Author

@derailed It's been quite some time... Could you get this merged?

@mih-kopylov
Copy link

@derailed I'm looking for this feature for quite a while. Could you please facilitate to get this PR merged?

@yeahdongcn
Copy link
Contributor Author

Code rebased from upstream/main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-tlc Pr needs additional updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants