Merge lp://qastaging/~gesha/linaro-license-protection/1146220 into lp://qastaging/~linaro-automation/linaro-license-protection/trunk
Proposed by
Georgy Redkozubov
Status: | Merged |
---|---|
Approved by: | Stevan Radaković |
Approved revision: | 186 |
Merged at revision: | 184 |
Proposed branch: | lp://qastaging/~gesha/linaro-license-protection/1146220 |
Merge into: | lp://qastaging/~linaro-automation/linaro-license-protection/trunk |
Diff against target: |
131 lines (+85/-0) 2 files modified
scripts/publish_to_snapshots.py (+31/-0) tests/test_publish_to_snapshots.py (+54/-0) |
To merge this branch: | bzr merge lp://qastaging/~gesha/linaro-license-protection/1146220 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stevan Radaković | Approve | ||
Review via email: mp+160352@code.qastaging.launchpad.net |
Description of the change
This branch reinstates BUILD-INFO.txt support for publishing service.
Publishing script checks if BUILD-INFO.txt is present among artifacts being published or among already published artifacts for the same build. Otherwise publishing is aborted with an error.
To post a comment you must log in.
Nice work gesha, thanks!
Only thing I dislike here is having return values(PASS, FAIL) in the method instead of using the exceptions as this is a very good example on when they should be used. That way you don't need to return anything at all, just catch exception if the method throws it. Tests would need to go through a little change as well because of it.