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

k9s: add aliases, plugins, views #4627

Closed
wants to merge 1 commit into from

Conversation

katexochen
Copy link
Contributor

@katexochen katexochen commented Nov 2, 2023

Description

Adding the remaining config files for k9s. Documented at https://k9scli.io/

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all or nix develop --ignore-environment .#all using Flakes.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.

Maintainer CC

@liyangau

Copy link
Member

@sumnerevans sumnerevans left a comment

Choose a reason for hiding this comment

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

just nits, otherwise looks good

modules/programs/k9s.nix Outdated Show resolved Hide resolved
modules/programs/k9s.nix Outdated Show resolved Hide resolved
modules/programs/k9s.nix Outdated Show resolved Hide resolved
Adding the remaining config files of k9s that weren't covered yet.

Signed-off-by: Paul Meyer <49727155+katexochen@users.noreply.github.com>
rycee pushed a commit that referenced this pull request Nov 11, 2023
Adding the remaining config files of k9s that weren't covered yet.

PR #4627
rycee pushed a commit that referenced this pull request Nov 11, 2023
Adding the remaining config files of k9s that weren't covered yet.

PR #4627
@rycee
Copy link
Member

rycee commented Nov 11, 2023

Thanks for the contribution! I rebased it to master in 691cbcc 🙂

@rycee rycee closed this Nov 11, 2023
@katexochen katexochen deleted the k9s/allfiles branch November 11, 2023 07:21
gkleen pushed a commit to gkleen/home-manager that referenced this pull request Dec 9, 2023
Adding the remaining config files of k9s that weren't covered yet.

PR nix-community#4627
fufexan pushed a commit to fufexan/home-manager that referenced this pull request Feb 24, 2024
Adding the remaining config files of k9s that weren't covered yet.

PR nix-community#4627
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.

3 participants