Skip to content

chore(ci): keep report uploads from failing on artifact quota - #25

Merged
dannysteenman merged 1 commit into
mainfrom
ci/tolerate-artifact-quota
Oct 13, 2026
Merged

dannysteenman merged 1 commit into
mainfrom
ci/tolerate-artifact-quota

Conversation

@dannysteenman

Copy link
Copy Markdown
Member

What

Adds step-level continue-on-error: true to the report-only actions/upload-artifact step in CI.

Why

The account-wide GitHub artifact storage quota was hit ("Unable to upload any new artifacts"), which failed otherwise-green jobs. Nothing downloads these report artifacts, so a failed upload should not fail the job.

Hand-off uploads that a later job downloads (for example the release VSIX) are deliberately unchanged and still fail loudly.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T13:34:03.833235Z 2b9ede1 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@dannysteenman dannysteenman added the impact:limited Two-way door: a revert fully undoes it, including anything already deployed or published label Oct 13, 2026
@dannysteenman
dannysteenman merged commit 3d3c1c7 into main Oct 13, 2026
4 checks passed
@dannysteenman
dannysteenman deleted the ci/tolerate-artifact-quota branch October 13, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:limited Two-way door: a revert fully undoes it, including anything already deployed or published

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant