wfxr / forgit

:zzz: A utility tool powered by fzf for using git interactively.
MIT License
4.33k stars 136 forks source link

Diffs for Python files have somewhat nonsensical headings if they are at the module level namespace #202

Closed taranlu-houzz closed 2 years ago

taranlu-houzz commented 2 years ago

Check list

Environment info

Problem / Steps to reproduce

If there are changes that occur outside of a function or class, the "Header" that forgit choses for that particular diff chunk is often confusingly unrelated to the diff content. For example, the "Header" might be a random import line: Screen Shot 2022-05-18 at 02 07 00

It seems like maybe it would be better to just have a catchall term used for module level changes?

wfxr commented 2 years ago

@taranlu-houzz forgit has nothing to do with the diff content. It's provided by delta. Maybe You should probably ask on delta's repo.

taranlu-houzz commented 2 years ago

@wfxr Ah, right, of course. Sorry about that. I will ask on their repo. I forgot that I had installed delta to test it out a while back. Thank you for letting me know.