Skip to content

Remove artifact uploads on push, generate APK only on release#10

Merged
Cross2pro merged 2 commits into
masterfrom
copilot/update-github-action-for-apk
Nov 12, 2025
Merged

Remove artifact uploads on push, generate APK only on release#10
Cross2pro merged 2 commits into
masterfrom
copilot/update-github-action-for-apk

Conversation

Copilot AI commented Nov 12, 2025

Copy link
Copy Markdown

Artifacts were being generated on every push, wasting storage. APK should only be packaged when creating releases via tag push.

Changes

  • Removed artifact upload step that ran on every build
  • Restricted version extraction to tag pushes only (was running on all builds)
  • Updated documentation to reflect artifacts are no longer generated on push

Workflow Behavior

Before:

# Ran on every push
- name: Upload signed APK artifact
  if: steps.signing.outputs.signing_ready == 'true'
  uses: actions/upload-artifact@v4

After:

  • Regular pushes: build validation only, no artifacts
  • Tag pushes: create release with APK directly attached

Usage

# Daily development - no APK generated
git push origin main

# Release - APK attached to GitHub release
git tag v4.5 && git push origin main v4.5

APK files in releases are directly downloadable (not zipped like artifacts).

Original prompt

修改github action,不需要每次提交都生成artifacts,我只需要在创建release的时候能打包apk就行。而且现在还是会上传zip文件而不是apk文件,我希望直接创建apk文件以便于下载


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI self-assigned this Nov 12, 2025
Co-authored-by: Cross2pro <30151531+Cross2pro@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub Action to package APK on release only Remove artifact uploads on push, generate APK only on release Nov 12, 2025
Copilot AI requested a review from Cross2pro November 12, 2025 02:10
@Cross2pro Cross2pro marked this pull request as ready for review November 12, 2025 04:17
@Cross2pro Cross2pro merged commit e566441 into master Nov 12, 2025
3 checks 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.

2 participants