diff options
author | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-07-20 15:57:08 +0200 |
---|---|---|
committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-07-20 16:22:18 +0200 |
commit | dcc0cffbcf51f14388515fd9095848952d46db39 (patch) | |
tree | d09b43714623b326d3068cfa1ca181dbdb006d11 /src/rpc | |
parent | c575260f80e94026204f97714ba6bcb26237d818 (diff) | |
parent | 5c613aadd64453c75cb2373c6fcc1326c3cf0b7a (diff) |
Merge #13695: lint: Add linter for circular dependencies
5c613aadd64453c75cb2373c6fcc1326c3cf0b7a lint: Add linter for circular dependencies (Ben Woosley)
Pull request description:
Protects against added circular depencies, makes it explicit in the
code when circular dependencies have been removed.
Modeled after EXPECTED_BOOST_INCLUDES in lint-includes.sh
Example output:
```
$ test/lint/lint-circular-dependencies.sh
A new circular dependency in the form of "qt/paymentserver -> qt/walletmodel -> qt/paymentserver" appears to have been introduced.
$ echo $?
1
$ test/lint/lint-circular-dependencies.sh
Good job! The circular dependency "Fake" is no longer present.
Please remove it from EXPECTED_CIRCULAR_DEPENDENCIES in test/lint/lint-circular-dependencies.sh
to make sure this circular dependency is not accidentally reintroduced.
$ echo $?
1
$ test/lint/lint-circular-dependencies.sh
$ echo $?
0
```
Tree-SHA512: 4519434de29f6d50859daed1480e531c01c1cdbc3f0a5f093251daf62ae2b5b9073fb274b86f541a985e06837aa1165b76558c5f35fb51a759d72e83f1b61e44
Diffstat (limited to 'src/rpc')
0 files changed, 0 insertions, 0 deletions