Skip to content

fix(ci): stop reporting healthy dump runs as failed - #62

Merged
Seungpyo1007 merged 1 commit into
mainfrom
fix/ci-false-failures
Sep 15, 2026
Merged

Seungpyo1007 merged 1 commit into
mainfrom
fix/ci-false-failures

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member
  • refresh-data: pack the dump into one tar.gz before upload-artifact (crashed with Maximum call stack size exceeded on ~1M files, run 34923333120).
  • dump-refresh: create-pull-request gets continue-on-error; a new step fails the job only if no open PR exists for the branch. Run 34923332763 failed with diff is taking too long to generate although TechAPI #180 was created.

Refs #1

refresh-data: upload-artifact crashed with "Maximum call stack size
exceeded" on the ~1M-file dump; it now uploads one tar.gz.

dump-refresh: create-pull-request could fail with GitHub's "diff is
taking too long to generate" after the PR already existed (TechAPI
#180). That step now continues, and a follow-up fails the job only
when no open PR exists for the branch.

Refs #1
@Seungpyo1007
Seungpyo1007 merged commit 93a2af1 into main Sep 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant