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
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.
Revision history for this message
Ugo Riboni (uriboni) wrote :

Looks ok, if jenkins approves feel free to top-approve it yourself.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
165. By Sergio Schvezov

Adding missing files

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches