PR#740: Add notifications to PR - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

pagure

#740 Add notifications to PR

Merged Feb 19, 2016 by pingou. Opened Feb 16, 2016 by pingou.

pr_notifications into master

Download 740.patch

no initial comment

pingou commented Feb 16, 2016

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

ralph commented Feb 16, 2016

:+1: here.

ralph commented Feb 16, 2016

No tests?

pingou commented Feb 17, 2016

There are some tests: https://pagure.io/pagure/pull-request/740#_9,5-11

pingou commented Feb 18, 2016

@ralph do you want more tests or is it fine as such?

ralph commented Feb 19, 2016

Oh, I missed them. :+1: here.

pingou commented Feb 19, 2016

Thanks for the review!

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

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

© Red Hat, Inc. and others.