Add support to GitCmd

[Aurze]

Goal

I wanna use difftastic to see PKGBUILD files diff

Solution

I allow to specify the git command to use git -C {{repo_path}} diff --ignore-space-change --ignore-all-space {{last_install_hash}} {{current_hash}} -- .
It allso has some "params" for variables.

Originally I implemented a "GitDiffExternalTool" but then I realize any new feature we would need yet an other params so instead I went with something more generic

Looking for feedback

So this solution works, I didn't test it like crazy yet, I'm looking on feedback like is this okay to let the user specify the full git command? This also deprecate the GitDiffArgs param, I could also put it as a replacable var? Or I can go back to a more specific approach of a GitDiffExternalTool.

[actionless]

(and idea itself is good)

[actionless]

thanks!

[actionless]

please also rebase on top of master branch, to make the CI pass

[actionless]

now you need to resolve the actual codestyle mismatches CI found:

https://github.com/actionless/pikaur/actions/runs/33690974456/job/100483572792?pr=916

[actionless]

CI is still failing

[actionless]

hopefully

you know, you could run CI locally? 😺

https://github.com/actionless/pikaur/blob/master/README.md#code

[Aurze]

:face_palme: I miss the Contributing section.
Okay so now lint pass on my side but for ./maintenance_scripts/docker_test.sh it says you can use "all" as a module but when I do so it says module not found. So I ran a bunch manually. If I miss some, sorry. But is there a way to run all test?

[actionless]

for ./maintenance_scripts/docker_test.sh it says you can use "all" as a module but when I do so it says module not found

that's weird - it must work that way, what was the exact cli args?

[coveralls]

Coverage Report for CI Build 34435175912

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+0.2%) to 83.321%

Details

• Coverage increased (+0.2%) from the base build.
• Patch coverage: 1 of 1 lines across 1 file are fully covered (100%).
• 148 coverage regressions across 4 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

148 previously-covered lines in 4 files lost coverage.

File	Lines Losing Coverage	Coverage
pikaur/install_cli.py	88	73.96%
pikaur/privilege.py	29	32.84%
pikaur/args.py	28	89.78%
pikaur/help_cli.py	3	91.43%

---

Coverage Stats

Relevant Lines:	5510
Covered Lines:	4591
Line Coverage:	83.32%
Coverage Strength:	0.83 hits per line

---

💛 - Coveralls

[actionless]

i was about to merge but whilst writing commit message i've realized the problem with naming - it turns GitDiffArgs into GitCmd, instead of GitDiffCmd - which is pretty confusing

[Aurze]

Agreed, I renamed the config

[actionless]

thanks for your contribution!