pagure

#751 Show context in outdated comments
Closed: Fixed by wombelix. Opened by puiterwijk.

Closed: Fixed
Reopen Issue

It would be great if one could also see the context for outdated comments (as discussed in #744).
This could be implemented by storing the tip of the tree + filepath at the moment of comment creation, so you have the history of the file at that point.

We should probably start storing this info as soon as possible, even if we don't display the context yet, so we have the data for when we do for as many PRs as possible.

Started at: https://pagure.io/pagure/pull-request/754

Another issue for this is what if the commits are squashed and force pushed? the tree + filepath is going to not work for showing the context there, right?

We may need to cache it in the DB

I would like to see the context for non-outdated comments too, fwiw.

+1

Started at: https://pagure.io/pagure/pull-request/754

Related PR merged.

Metadata Update from @wombelix:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)
Metadata
None
None
None
None
None
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.