Merge lp://staging/~sergiusens/notes-app/testing_artifacts into lp://staging/notes-app
Proposed by
Sergio Schvezov
Status: | Merged |
---|---|
Approved by: | Sergio Schvezov |
Approved revision: | 165 |
Merged at revision: | 166 |
Proposed branch: | lp://staging/~sergiusens/notes-app/testing_artifacts |
Merge into: | lp://staging/notes-app |
Diff against target: |
334 lines (+286/-1) 6 files modified
CMakeLists.txt (+20/-0) cmake/EnableCoverageReport.cmake (+153/-0) cmake/FindLcov.cmake (+29/-0) cmake/Findgcovr.cmake (+31/-0) cmake/ParseArguments.cmake (+52/-0) tests/unittests/CMakeLists.txt (+1/-1) |
To merge this branch: | bzr merge lp://staging/~sergiusens/notes-app/testing_artifacts |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot | continuous-integration | Approve | |
Ugo Riboni (community) | Approve | ||
Review via email: mp+177351@code.staging.launchpad.net |
Commit message
Adding test xml output and coverage targets.
Description of the change
Adding test xml output and coverage build targets.
More than anything, to get reports that can be processed (i.e.; Jenkins)
To post a comment you must log in.
Looks ok, if jenkins approves feel free to top-approve it yourself.