lint: enable suspicious-semicolon_rule by default

chipsalliance · Dec 21, 2023 · 874b13a · 874b13a
1 parent 4e8133d
commit 874b13a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/verilog/analysis/default_rules.h b/verilog/analysis/default_rules.h
@@ -65,6 +65,7 @@ inline constexpr const char* kDefaultRuleSet[] = {
     "positive-meaning-parameter-name",
     "constraint-name-style",
     "suggest-parentheses",
+    "suspicious-semicolon",
     "truncated-numeric-literal",
     // TODO(fangism): enable in production:
     // TODO(b/155128436): "uvm-macro-semicolon"

diff --git a/verilog/tools/lint/BUILD b/verilog/tools/lint/BUILD
@@ -69,7 +69,7 @@ _linter_test_configs = [
     ("signal-name-style", "signal_name_style", False),
     ("struct-union-name-style", "struct_name_style", True),
     ("struct-union-name-style", "union_name_style", True),
-    ("suspicious-semicolon", "suspicious_semicolon", False),
+    ("suspicious-semicolon", "suspicious_semicolon", True),
     ("v2001-generate-begin", "generate_begin_module", True),
     ("void-cast", "void-cast", True),
     ("undersized-binary-literal", "undersized_binary_literal", True),