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

Provide kubel-kill-buffer command. #134

Merged
merged 1 commit into from
Aug 21, 2024
Merged

Conversation

graywolf
Copy link
Contributor

Since C-c C-k is fairly popular binding (magit, message, ...), provide it as well. It allows quickly canceling the edit without applying the changes.

This also changes kubel-yaml-editing-mode to be defined using define-derived-mode from yaml-mode, with standalone map in kubel-yaml-editing-map.

  • kubel.el (kubel-kill-buffer-query): New custom.
    (kubel-kill-buffer): New function.
    (kubel-yaml-editing-map): New variable.
    (kubel-yaml-editing-mode): Change to derived-mode from minor-mode.
    (kubel--describe-resource, kubel-get-resource-details): Stop calling yaml-mode.

Since C-c C-k is fairly popular binding (magit, message, ...), provide it as
well.  It allows quickly canceling the edit without applying the changes.

This also changes kubel-yaml-editing-mode to be defined using
define-derived-mode from yaml-mode, with standalone map in
kubel-yaml-editing-map.

* kubel.el (kubel-kill-buffer-query): New custom.
(kubel-kill-buffer): New function.
(kubel-yaml-editing-map): New variable.
(kubel-yaml-editing-mode): Change to derived-mode from minor-mode.
(kubel--describe-resource, kubel-get-resource-details): Stop calling yaml-mode.
Copy link
Owner

@abrochard abrochard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@abrochard abrochard merged commit a3f3fc9 into abrochard:master Aug 21, 2024
5 checks passed
@graywolf
Copy link
Contributor Author

Uff, I made a mistake and you were too quick. Please see #136 .

@graywolf
Copy link
Contributor Author

Really sorry about this.

@abrochard
Copy link
Owner

No worries! I should have caught it. Thank you for the follow up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants