Skip to content

Commit 2f9cfa9

Browse files
committed
Lintcheck: Fix Errors, because of course
1 parent 10bf729 commit 2f9cfa9

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

.github/workflows/lintcheck.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -119,10 +119,10 @@ jobs:
119119
# https://docs.github.com/actions/using-workflows/workflow-commands-for-github-actions#adding-a-markdown-summary
120120
# That's why we first log to file and then to the summary and logs
121121
run: |
122-
./target/debug/lintcheck diff {base,head}/ci_crates_logs.json >> full_diff.md
123122
./target/debug/lintcheck diff {base,head}/ci_crates_logs.json --truncate >> truncated_diff.md
124123
head -c 1024000 truncated_diff.md >> $GITHUB_STEP_SUMMARY
125124
cat truncated_diff.md
125+
./target/debug/lintcheck diff {base,head}/ci_crates_logs.json >> full_diff.md
126126
127127
- name: Upload full diff
128128
uses: actions/upload-artifact@v4

lintcheck/src/json.rs

+5
Original file line numberDiff line numberDiff line change
@@ -79,9 +79,14 @@ pub(crate) fn diff(old_path: &Path, new_path: &Path, truncate: bool) {
7979

8080
let lint_warnings = group_by_lint(added, removed, changed);
8181

82+
8283
print_summary_table(&lint_warnings);
8384
println!();
8485

86+
if lint_warnings.is_empty() {
87+
return;
88+
}
89+
8590
let truncate_after = if truncate {
8691
// Max 15 ensures that we at least have five messages per lint
8792
DEFAULT_LIMIT_PER_LINT

0 commit comments

Comments
 (0)