Page MenuHomePhabricator

[cmake] check-security => security-check
ClosedPublic

Authored by deadalnix on Mar 6 2020, 16:32.

Details

Summary

All the check-whatever targets are unit tests with the exception of check-security and check-symbol (which I intend to port over too). The script is named security-check.py and the function called by cmake as well. This is just more consistent all over this way.

Bonus: Use the COMMENT facility rather than echoing stuff on the command line when runnign the checks.

Test Plan
ninja all check security-check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
securitycheck
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9751
Build 17382: Default Diff Build & Tests
Build 17381: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Mar 6 2020, 16:58
This revision was automatically updated to reflect the committed changes.