Skip to content

Commit e5aac8a

Browse files
fix(clang-tidy): fix issues it found in CI
Curiously, the changed includes seem to allow `clang-tidy` to see issues that it previously didn't see. This commit fixes them. Signed-off-by: Ingo Müller <ingomueller@google.com>
1 parent 98e5091 commit e5aac8a

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

include/substrait/expression/DecimalLiteral.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
#include <utility>
88

99
namespace substrait::proto {
10-
class Expression_Literal_Decimal;
10+
class Expression_Literal_Decimal; // NOLINT
1111
}
1212

1313
namespace io::substrait::expression {

src/substrait/textplan/parser/SubstraitPlanRelationVisitor.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@
1919

2020
namespace substrait::proto {
2121
class Expression;
22-
class Expression_Literal;
23-
class Expression_Literal_Map_KeyValue;
24-
class Type_Struct;
22+
class Expression_Literal; // NOLINT
23+
class Expression_Literal_Map_KeyValue; // NOLINT
24+
class Type_Struct; // NOLINT
2525
} // namespace substrait::proto
2626

2727
namespace io::substrait::textplan {

src/substrait/textplan/parser/SubstraitPlanSubqueryRelationVisitor.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@
1919

2020
namespace substrait::proto {
2121
class Expression;
22-
class Expression_Literal;
23-
class Expression_Literal_Map_KeyValue;
24-
class Type_Struct;
22+
class Expression_Literal; // NOLINT
23+
class Expression_Literal_Map_KeyValue; // NOLINT
24+
class Type_Struct; // NOLINT
2525
} // namespace substrait::proto
2626

2727
namespace io::substrait::textplan {

0 commit comments

Comments
 (0)