pagure

#1626 Couple of fixes to the blame feature
Merged by pingou. Opened by pingou.
fix_blame_view  into  master

Download 1626.patch

  • Fix displaying git blame when the file contains some non-ascii characters

  • Fix accessing repo where the head is unborned

rebased

1 new commit added

  • Handle the situation where the repo's head is unborned

I think the best place to decode this is when the file is loaded out of git. I think that happens here. That way you never have to worry about encoding here.

I only have the one comment. I think you can use that encoding_utils module to guess the encoding type and then decode it, since it's not necessary utf-8 encoded.

1 new commit added

  • Decode the blamed file before rendering the template

:thumbsup: I'm happy if you're happy

Local tests are working, so I am as well :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.