HomePhabricator

[linter] add exceptions to include guards and file name linters

Description

[linter] add exceptions to include guards and file name linters

Summary:
an upcoming backport (https://github.com/bitcoin/bitcoin/pull/17009/commits/bebb637472d0469037a9f438572fc71db4236d97) regarding Fuzz harnesses adds a header file from the Chromium project. this lets us add exceptions as to avoid modifying it


Depends on D6741

Test Plan:

cd src
mv hash.h Hash.h
vim Hash.h

run :%s/BITCOIN_HASH_H/TEST_LINTER_H/g and :wq!

git add Hash.h

arc lint

linter yells at you

add "Hash.h" to the EXCEPTIONS array in both FileNameLinter.php and IncludeGuardLinter.php

arc lint

linter no longer yells at you

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D6744

Details

Provenance
majcostaAuthored on Jun 25 2020, 22:08
majcostaPushed on Jul 1 2020, 18:34
Reviewer
Restricted Project
Differential Revision
D6744: [linter] add exceptions to include guards and file name linters
Parents
rABC9e0128d08619: [backport#17076] tests: Add fuzzing harness for CheckTransaction(...)…
Branches
Unknown
Tags
Unknown