Skip to content

Commit 88e4e11

Browse files
authored
Merge pull request #4065 from ashnaaseth2325-oss/fix/genmetrics-required-args
fix: require --logs, --reports, and --results to prevent NoneType crash in genMetrics
2 parents 8d0f94b + f094e24 commit 88e4e11

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

flow/util/genMetrics.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ def parse_args():
4848
"--output", "-o", required=False, default="metadata.json", help="Output file"
4949
)
5050
parser.add_argument("--hier", "-x", action="store_true", help="Hierarchical JSON")
51-
parser.add_argument("--logs", help="Path to logs")
52-
parser.add_argument("--reports", help="Path to reports")
53-
parser.add_argument("--results", help="Path to results")
51+
parser.add_argument("--logs", required=True, help="Path to logs")
52+
parser.add_argument("--reports", required=True, help="Path to reports")
53+
parser.add_argument("--results", required=True, help="Path to results")
5454
args = parser.parse_args()
5555

5656
return args

0 commit comments

Comments
 (0)