-
Notifications
You must be signed in to change notification settings - Fork 14
Restore linters missed when merging bitcoin 0.17 #1101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Julian Fleischer <julian@thirdhash.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 42621d0
Signed-off-by: Julian Fleischer <julian@thirdhash.com>
lint-clang-format is used for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ConceptACK 3fcad73
Since bitcoin 0.17 all linters reside in |
Signed-off-by: Julian Fleischer <julian@thirdhash.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 2147124
Signed-off-by: Julian Fleischer <julian@thirdhash.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 025ffd0
This pull request also executes all linters always now (instead of exiting on first erroring script). Backporting to bitcoin in bitcoin/bitcoin#16036 |
The dtr-org#1101 broke pre-commit git hook by moving linter from `contrib/devtools` to`test/lint` directory. This commit fixes that. Signed-off-by: Stanislav Frolov <stanislav@thirdhash.com>
The #1101 broke pre-commit git hook by moving linter from `contrib/devtools` to`test/lint` directory. This commit fixes that. Signed-off-by: Stanislav Frolov <stanislav@thirdhash.com>
Signed-off-by: Julian Fleischer julian@thirdhash.com