Skip to content

feat(git): add renamed support#88

Open
PFiS1737 wants to merge 4 commits intoyazi-rs:mainfrom
PFiS1737:feat-renamed
Open

feat(git): add renamed support#88
PFiS1737 wants to merge 4 commits intoyazi-rs:mainfrom
PFiS1737:feat-renamed

Conversation

@PFiS1737
Copy link
Contributor

This is the initial implementation of renamed support. Note that I removed :args(paths) and made the git status command always run at the repository root. This causes fetcher to process the entire git status output every time it runs, which will obviously lead to performance issues.

However, it seems we have no other choice: When using :args(paths), the command never outputs renamed files, and since renames can occur across folders, it must be executed at the root directory.

We might consider adding an option to let users decide whether to enable this feature, even at the cost of performance.

@PFiS1737
Copy link
Contributor Author

adding an option to let users decide whether to enable this feature

How can we get opts in fetch() context? Using a global variable?

@PFiS1737
Copy link
Contributor Author

PFiS1737 commented May 2, 2025

Waiting for #111 to be merged. (the 09df50f commit)

@sxyazi sxyazi force-pushed the main branch 2 times, most recently from 1ff5f8b to 4df8f70 Compare May 26, 2025 12:12
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.

1 participant