Skip to content

Commit 5c308a5

Browse files
ctruedenclaude
andcommitted
pombast CI: fix publish, add log artifacts
publish.sh checked `git diff --quiet` after staging the files with `git add`, so it always saw a clean working tree and reported "No changes" -- nothing was ever committed to status.scijava.org's gh-pages branch. Check staged changes with `git diff --cached --quiet` instead. Also upload target/pombast logs (mega-melt validation + per-component build logs) and the generated reports as workflow artifacts, so melt and smelt failures can be diagnosed without waiting on a Pages deploy, and set COLUMNS so the Rich tables in the CI logs are legible. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 10f171f commit 5c308a5

3 files changed

Lines changed: 33 additions & 1 deletion

File tree

.github/publish.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,9 @@ do
3030
done
3131

3232
cd "$dest_dir"
33-
if git diff --quiet .
33+
# Check staged changes (--cached): the files above were already `git add`ed, so
34+
# a plain `git diff` would see an always-clean working tree and never commit.
35+
if git diff --cached --quiet
3436
then
3537
echo "== No changes =="
3638
else

.github/workflows/build.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,22 @@ jobs:
7171
- name: Run pombast checks (melt + smelt)
7272
run: bin/check.sh
7373
shell: bash
74+
env:
75+
COLUMNS: '200' # widen Rich tables so log output is legible
76+
77+
- name: Upload pombast logs
78+
if: always()
79+
uses: actions/upload-artifact@v4
80+
with:
81+
name: pombast-logs
82+
# mega-melt validation.log + dependency-tree.log + per-component build
83+
# logs, plus the smelt report. Lets us diagnose melt/smelt failures
84+
# without waiting on a status.scijava.org deploy.
85+
path: |
86+
target/pombast/**/*.log
87+
target/pombast/smelt.json
88+
if-no-files-found: warn
89+
retention-days: 14
7490

7591
- name: Delete old pombast cache
7692
if: always()

.github/workflows/status.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,20 @@ jobs:
6464
shell: bash
6565
env:
6666
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
67+
COLUMNS: '200' # widen Rich tables so log output is legible
68+
69+
- name: Upload status reports
70+
if: always()
71+
uses: actions/upload-artifact@v4
72+
with:
73+
name: status-reports
74+
path: |
75+
target/pombast/index.html
76+
target/pombast/badges.json
77+
target/pombast/team.html
78+
target/pombast/team.json
79+
if-no-files-found: warn
80+
retention-days: 14
6781

6882
- name: Delete old pombast status cache
6983
if: always()

0 commit comments

Comments
 (0)