Repository navigation
repo.blame should allow passing None as the rev parameter. #1835
Description
Activity
Thanks for sharing your finding!
It does seems to make sense to adjust the annotations to explicitly allow this use-case.
I'll gladly make a pull request with the fix if you or other contributors don't have anything more to say about this issue.
Are there any other places in the code that comes to mind for which such a fix could also make sense? The only other git command that I use without specifiying a revision is
git diff, but theDiffableinterface seems to already support passingNoneas theotherparameter.Reacted by Sebastian ThielThanks for offering! It's OK to keep the PR focussed on the problem at hand.
Reacted by Gaubbe
This issue only affects the type hint given to the
revparameter, as passingNoneto the function does work and gives the expected behavior, which is running thegit blame -p -- <file>command without a revision, which gives back the blame including changes to the local working directory that haven't been committed yet.While the code works at runtime, language servers such as pylance give back an error, as
Noneis not a valid type according to the type hints.