Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor: remove -u option from git status in forgit add #397

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

wfxr
Copy link
Owner

@wfxr wfxr commented Aug 14, 2024

Check list

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation

Description

I found that all untracked files will always be listed recursively after forgit add is called. This is annoying when there are some untracked directories with lots of files in a repo. Useful information will be flushed out by these untracked files. This patch removed the -u option from git status to just list the untracked directory itself.

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Breaking change
  • Documentation change

Test environment

  • Shell
    • bash
    • zsh
    • fish
  • OS
    • Linux
    • Mac OS X
    • Windows
    • Others:

Signed-off-by: Wenxuan Zhang <wenxuangm@gmail.com>
@wfxr wfxr force-pushed the ga-minor-change branch from 363720b to dad03c3 Compare August 14, 2024 02:39
@wfxr wfxr requested review from carlfriedrich, sandr01d and cjappl and removed request for carlfriedrich and sandr01d August 14, 2024 02:50
Copy link
Collaborator

@sandr01d sandr01d left a comment

Choose a reason for hiding this comment

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

Works for me

@wfxr wfxr merged commit 8b60a89 into master Aug 15, 2024
7 checks passed
@wfxr wfxr deleted the ga-minor-change branch August 15, 2024 02:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants