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

feat: add git alias and auto completion #506

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

waitingsong
Copy link
Contributor

@waitingsong waitingsong commented Feb 5, 2025

Add alias and the auto completion:

  • ga for git add
  • gb for git branch
  • gc for git checkout
  • gl for git log
  • glg for git log --graph --oneline --decorate --date=short --pretty=format:"%C(yellow)%h%Creset %Cgreen%ad%Creset %Cblue%an%Creset %Cred%d%Creset %s"
  • gps for git push
  • gs for git switch

@waitingsong waitingsong force-pushed the git-auto-completion branch 3 times, most recently from b4346b0 to 4c048ee Compare February 6, 2025 05:52
Add alias and the auto completion:
- `ga` for `git add`
- `gb` for `git branch`
- `gc` for `git checkout`
- `gl` for `git log`
- `glg` for `git log --graph --oneline --decorate --date=short --pretty=format:"%C(yellow)%h%Creset %Cgreen%ad%Creset %Cblue%an%Creset %Cred%d%Creset %s"`
- `gps` for `git push`
- `gs` for `git switch`
@Vonng
Copy link
Member

Vonng commented Feb 14, 2025

I thought about it, and while adding too many aliases may be convenient, it can lead to confusion. A better approach would be to add a node_alias parameter, allowing users to pass in and define any alias they want to use. Therefore, this PR might be resolved in another way.

@Vonng
Copy link
Member

Vonng commented Feb 14, 2025

#511

@Vonng Vonng merged commit 6bf88cb into pgsty:main Feb 16, 2025
1 check passed
@waitingsong waitingsong deleted the git-auto-completion branch February 16, 2025 07:20
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