Skip to content

Conversation

@lstein
Copy link
Owner

@lstein lstein commented Jan 4, 2026

This pull request updates the deployment workflow to improve release tagging and artifact uploading automation. The main changes ensure that releases are consistently tagged, prevent duplicate tags, and pass the tag name through the workflow for artifact uploads.

Release Tagging Improvements

  • Added logic to output the release version and tag from the tag-release job, and to check if the tag already exists before creating a new one, preventing duplicate tags. (.github/workflows/deploy.yml) [1] [2]

Artifact Upload Automation

  • Modified the upload-release job to depend on the tag-release job and to pass the generated tag name as an input to the artifact upload workflow, ensuring artifacts are correctly attached to the release tag. (.github/workflows/deploy.yml, .github/workflows/upload-artifacts.yml) [1] [2]
  • Updated the artifact upload step to use the provided tag_name input when uploading release artifacts, ensuring they are associated with the correct release tag. (.github/workflows/upload-artifacts.yml)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the deployment workflow to add release tagging automation and improve artifact uploading by passing tag information between jobs.

  • Added job outputs to the tag-release job to expose version and tag information to downstream jobs
  • Implemented tag existence checking to prevent duplicate tags from being created
  • Modified the upload-release job to accept and use the tag name from the tag-release job

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.

File Description
.github/workflows/deploy.yml Added outputs to tag-release job, implemented tag existence check, added conditional tagging, and passed tag to upload-release job
.github/workflows/upload-artifacts.yml Added tag_name input parameter and configured artifact upload to use the provided tag

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

- name: Check if tag exists
id: check_tag
run: |
if git ls-remote --tags origin | grep -q "refs/tags/v${{ steps.get_version.outputs. version }}"; then
Copy link

Copilot AI Jan 4, 2026

Choose a reason for hiding this comment

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

There's a typo in the variable reference. There's an extra space between "steps.get_version.outputs." and "version". This should be "steps.get_version.outputs.version" without the extra space.

Copilot uses AI. Check for mistakes.
Copy link
Owner Author

Choose a reason for hiding this comment

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

@copilot open a new pull request to apply changes based on this feedback

Copy link
Contributor

Copilot AI commented Jan 4, 2026

@lstein I've opened a new pull request, #191, to work on those changes. Once the pull request is ready, I'll request review from you.

@lstein
Copy link
Owner Author

lstein commented Jan 4, 2026

@copilot Please repeat your review of this code to ensure that it is syntactically and semantically correct.

Copy link
Contributor

Copilot AI commented Jan 4, 2026

@lstein I've opened a new pull request, #194, to work on those changes. Once the pull request is ready, I'll request review from you.

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