pagure

#2171 Update the ticket metadata before adding the new comment (if there is one)
Merged by pingou. Opened by pingou.
fixes  into  master

Download 2171.patch

Fixes https://pagure.io/pagure/issue/2163

LGTM, I don't know if it is practical to test this so I think it's OK without adding tests. I recommend ensuring that you do have test coverage here though.

We have test coverage for this piece of the code already yes. Testing this won't be really practical, so I'll go ahead and merge this.

Maybe I can think of something, but it is clearly not something that is easily testable.

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.