greenwave

#285 Update Release Notes for 0.9.5
Merged by lholecek. Opened by lholecek.
lholecek/greenwave update-release-notes  into  master

Download 285.patch
no initial comment

:+1:

When did they stop behaving as expected? What were they doing instead?

rebased onto f6a88f8a9016ecc702c9ef49e1dcf5c944a495d3

When did they stop behaving as expected? What were they doing instead?

@dcallagh I've rephrased it:

Waivers with waived=false don't waive failed test results (this was broken in previous versions).

rebased onto 366dddaf77e74fe7ca9a623266184b2ba0de7181

Oh, was it always broken? Were we considering waived: false as if it were waived: true?

Oops :-)

Oh, was it always broken?

I don't think it was always broken, just with some recent refactoring and feature-adding. But since there were no tests it could've been broken for a long time.

Oops indeed :).

Commit 229b7485 fixes this pull-request

Pull-Request has been merged by lholecek

Pull-Request has been merged by lholecek
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.