Shahar Havivi has posted comments on this change.
Change subject: jobs: Remove unused method
..
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/51214
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: jobs: Remove unused method
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Dan Kenigsberg has submitted this change and it was merged.
Change subject: jobs: Remove unused method
..
jobs: Remove unused method
Nobody is calling Job.validate_done(), and I don't want to add test for
code that we don't
Dan Kenigsberg has posted comments on this change.
Change subject: jobs: Remove unused method
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/51214
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: jobs: Remove unused method
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:
Nir Soffer has posted comments on this change.
Change subject: jobs: Remove unused method
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/51214
To unsubscribe, visit https://gerrit.ovirt.org/settings
Nir Soffer has uploaded a new change for review.
Change subject: jobs: Remove unused method
..
jobs: Remove unused method
Nobody is calling Job.validate_done(), and I don't want to add test for
code that we don't need.