Skip to content

Commit 7ad731f

Browse files
committed
fix: use correct output variable names
In compliance with v1, the output names were missing the `clang-` prefix for `clang-tidy-checks-failed` and `clang-format-checks-failed`.
1 parent 2489719 commit 7ad731f

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

cpp-linter/src/rest_client.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -144,11 +144,11 @@ impl RestClient {
144144
value: format!("{}", format_checks_failed + tidy_checks_failed),
145145
},
146146
OutputVariable {
147-
name: "format-checks-failed".to_string(),
147+
name: "clang-format-checks-failed".to_string(),
148148
value: format_checks_failed.to_string(),
149149
},
150150
OutputVariable {
151-
name: "tidy-checks-failed".to_string(),
151+
name: "clang-tidy-checks-failed".to_string(),
152152
value: tidy_checks_failed.to_string(),
153153
},
154154
];
@@ -590,8 +590,8 @@ mod test {
590590
async fn check_comment_concerns() {
591591
let (comment, gh_out) = create_comment(false).await;
592592
assert!(&comment.contains(":warning:\nSome files did not pass the configured checks!\n"));
593-
let fmt_pattern = Regex::new(r"format-checks-failed=(\d+)\n").unwrap();
594-
let tidy_pattern = Regex::new(r"tidy-checks-failed=(\d+)\n").unwrap();
593+
let fmt_pattern = Regex::new(r"clang-format-checks-failed=(\d+)\n").unwrap();
594+
let tidy_pattern = Regex::new(r"clang-tidy-checks-failed=(\d+)\n").unwrap();
595595
for pattern in [fmt_pattern, tidy_pattern] {
596596
let number = pattern
597597
.captures(&gh_out)
@@ -614,7 +614,7 @@ mod test {
614614
assert!(comment.contains(":heavy_check_mark:\nNo problems need attention."));
615615
assert_eq!(
616616
gh_out,
617-
"checks-failed=0\nformat-checks-failed=0\ntidy-checks-failed=0\n"
617+
"checks-failed=0\nclang-format-checks-failed=0\nclang-tidy-checks-failed=0\n"
618618
);
619619
}
620620
}

0 commit comments

Comments
 (0)