Skip to content

CI: attach release artifacts - #20

Merged
Kyo-s-s merged 1 commit into
mainfrom
release-artifacts
Mar 11, 2026
Merged

Kyo-s-s merged 1 commit into
mainfrom
release-artifacts

Conversation

@Kyo-s-s

@Kyo-s-s Kyo-s-s commented Mar 11, 2026

Copy link
Copy Markdown
Owner

codex を信じてはいけない

@Kyo-s-s
Kyo-s-s requested a review from Copilot March 11, 2026 15:42
@Kyo-s-s
Kyo-s-s merged commit b6c7ae6 into main Mar 11, 2026
5 checks passed
@Kyo-s-s
Kyo-s-s deleted the release-artifacts branch March 11, 2026 15:44
@Kyo-s-s

Kyo-s-s commented Mar 11, 2026

Copy link
Copy Markdown
Owner Author

これ実際に release しないとテストできないじゃん! うぐぐぐぐ

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds CI support to attach the built extension zip as an asset to GitHub Releases, complementing the existing “main-latest” rolling prerelease flow.

Changes:

  • Trigger workflow on release.published events.
  • Add a publish-release job that downloads the build artifact, renames it to the versioned name, and uploads it to the corresponding GitHub Release.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

uses: softprops/action-gh-release@v2
with:
tag_name: ${{ github.event.release.tag_name }}
files: dist/${{ needs.build.outputs.zip_name }}

Copilot AI Mar 11, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

softprops/action-gh-release will error if an asset with the same name already exists on the release (e.g., when re-running this workflow). Consider setting overwrite_files: true (as done in publish-main-latest) or otherwise handling existing assets to make re-runs idempotent.

Suggested change
files: dist/${{ needs.build.outputs.zip_name }}
files: dist/${{ needs.build.outputs.zip_name }}
overwrite_files: true

Copilot uses AI. Check for mistakes.
Comment on lines +106 to +108
- name: Prepare release asset name
run: |
mv dist/chrome-mv3-prod.zip "dist/${{ needs.build.outputs.zip_name }}"

Copilot AI Mar 11, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This job assumes the downloaded artifact will contain a file named chrome-mv3-prod.zip (and then renames it). To reduce coupling to the internal filename inside the artifact, consider renaming/copying the packaged zip to ${{ steps.meta.outputs.zip_name }} in the build job before uploading, so downstream jobs can consume the artifact without hardcoding chrome-mv3-prod.zip.

Copilot uses AI. Check for mistakes.
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