Page MenuHomePhorge
Differential D326

Add option to not build tests
ClosedPublic
Actions

Authored by tusooa on Wed, Jul 8, 2:55 PM.
Tags
None
Referenced Files
F85611720: D326.1785111051.diff
Sat, Jul 25, 5:10 PM
F85611701: D326.1785107805.diff
Sat, Jul 25, 4:16 PM
F85611606: D326.1785088909.diff
Sat, Jul 25, 11:01 AM
F85609206: D326.1785051896.diff
Sat, Jul 25, 12:44 AM
Subscribers
None

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Commits
rK6980f5850a1d: Add option to not build tests
Summary

Build tests only if kazv_BUILD_TESTS is set to ON. This option is by default on so existing users will still get the same experience. Those who do not need tests can turn it off manually.

Type: skip

Test Plan

Check pipeline passes and it runs tests.

Diff Detail

Repository
rK kazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tusooa created this revision.Wed, Jul 8, 2:55 PM
Owners added a reviewer: O1: the Kazv Project.Wed, Jul 8, 2:55 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Wed, Jul 8, 3:53 PM
Harbormaster failed remote builds in B662: Diff 1016!
tusooa requested review of this revision.Wed, Jul 8, 3:55 PM

CI failure is not our fault

nannanko accepted this revision.Sat, Jul 11, 11:33 AM
This revision is now accepted and ready to land.Sat, Jul 11, 11:33 AM
This revision was landed with ongoing or failed builds.Sat, Jul 11, 7:11 PM
Closed by commit rK6980f5850a1d: Add option to not build tests. · Explain Why
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rK6980f5850a1d: Add option to not build tests.

Revision Contents
Changeset List

Diff 1022

View Options

CMakeLists.txt

Loading...
View Options

src/CMakeLists.txt

Loading...