Skip to content

CI: Publish OTA release on merged PRs to main and remove artifact upload#9

Merged
shai-almog merged 2 commits intomasterfrom
codex/remove-artifact-upload-and-change-ci-release
Feb 14, 2026
Merged

CI: Publish OTA release on merged PRs to main and remove artifact upload#9
shai-almog merged 2 commits intomasterfrom
codex/remove-artifact-upload-and-change-ci-release

Conversation

@shai-almog
Copy link
Contributor

Motivation

  • Replace per-job artifact uploads with a GitHub Release flow so full OTA content can be published automatically when a PR is merged to main.

Description

  • Changed workflow trigger to pull_request types: [closed] targeting main and gated the job with if: github.event.pull_request.merged == true so the job runs only for merged PRs.
  • Removed the actions/upload-artifact step and added a packaging step that archives all non-hidden files under OTA/ into dist/ota-${GITHUB_SHA}.tar.gz using find and tar (excluding dotfiles and hidden directories).
  • Added a release publishing step using softprops/action-gh-release@v2 which publishes the created tarball as a release asset and uses tag_name: ota-${{ github.sha }} and name: OTA Release ${{ github.sha }}.
  • Set workflow permission contents: write and updated checkout action to actions/checkout@v4.

Testing

  • Ran ./build_skins.sh locally; the script completed (exit code 0) but emitted expected warnings about missing theme files and some zip I/O messages in this checkout.
  • Verified the updated workflow file at .github/workflows/blank.yml and committed the change successfully.

Codex Task

@shai-almog shai-almog merged commit 2b36a5c into master Feb 14, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant