Skip to content

Unifica github release step#42

Merged
moniveloso merged 1 commit into
mainfrom
moni/unifica-github-release-step
May 26, 2026
Merged

Unifica github release step#42
moniveloso merged 1 commit into
mainfrom
moni/unifica-github-release-step

Conversation

@moniveloso
Copy link
Copy Markdown
Contributor

Context

The release workflow was failing due to GitHub release immutability constraints. The previous implementation created the release first and uploaded artifacts in a separate step afterward.

Previous flow:

  1. Create GitHub release
  2. Upload generated artifacts (dist/**)

With immutable releases enabled, modifying a release after creation can fail (HTTP 422), causing the workflow to break.

Changes

Updated the release workflow to upload artifacts during release creation instead of using a separate upload step.

Before:

gh release create <tag>
gh release upload <tag> dist/**

Copilot AI review requested due to automatic review settings May 25, 2026 18:28
Copy link
Copy Markdown

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

Este PR ajusta o workflow de release para evitar falhas causadas pela imutabilidade de GitHub Releases, passando a anexar os artefatos no momento da criação da release (em vez de fazer upload em um passo separado).

Changes:

  • Atualiza o comando gh release create para incluir os arquivos de dist/** como assets da release.
  • Remove o passo separado de gh release upload, unificando a criação da release e o upload de artefatos em uma única etapa.

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

@moniveloso moniveloso requested a review from a team May 25, 2026 18:29
@moniveloso moniveloso merged commit 0d155ca into main May 26, 2026
17 checks passed
@moniveloso moniveloso deleted the moni/unifica-github-release-step branch May 26, 2026 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants