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

Command alias implementation #9127

Closed
wants to merge 5 commits into from
Closed

Command alias implementation #9127

wants to merge 5 commits into from

Conversation

vicky1999
Copy link
Contributor

@vicky1999 vicky1999 commented Oct 9, 2023

Motivation

  • Command alias implemented for sub commands
  • If a command need to be aliased, we need to add the alias and the command to aliases map in main.cc.
  • The key should be combined with a # character

Related to #8914 #8875

Screenshots:

image

Description

  • I have created an alias with store ping to store info. After adding the alias in aliases map, the application was able to recognize both the commands and giving the same result

@vicky1999 vicky1999 requested a review from edolstra as a code owner October 9, 2023 11:27
@github-actions github-actions bot added the new-cli Relating to the "nix" command label Oct 9, 2023
@vicky1999
Copy link
Contributor Author

@edolstra @fricklerhandwerk
I have implemented the command level aliasing as per the requirement. Kindly review the changes.

@Ericson2314
Copy link
Member

We found a simple-stupid solution for nix store ping -> nix store info in #8975 (comment). Let's use that to finish off that PR, and then return to this?

@vicky1999 vicky1999 closed this Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-cli Relating to the "nix" command
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants