Skip to content

Conversation

vmaerten
Copy link
Member

@vmaerten vmaerten commented Aug 6, 2025

The nightly release was published as draft
Also the title was incorrect.
I've made the choice use another yaml config to not impact our "real" release workflow

Copy link

@Copilot 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 fixes issues with the nightly release configuration by creating a separate GoReleaser configuration file for nightly builds. The main issues addressed are preventing nightly releases from being published as drafts and ensuring correct release titles.

  • Created a dedicated .goreleaser-nightly.yml configuration file for nightly releases
  • Removed nightly-specific configuration from the main .goreleaser.yml file
  • Updated GitHub Actions workflows to use appropriate configurations and flags

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.goreleaser.yml Removed release draft setting and nightly configuration to clean up main release config
.goreleaser-nightly.yml New dedicated configuration for nightly releases with proper title template and settings
.github/workflows/release.yml Added --draft flag to ensure regular releases are published as drafts
.github/workflows/release-nightly.yml Updated to use the new nightly-specific configuration file

@vmaerten vmaerten force-pushed the fix/publish-nightly branch 2 times, most recently from 2782cf2 to 5703aa1 Compare August 6, 2025 19:44
@vmaerten vmaerten force-pushed the fix/publish-nightly branch from 5703aa1 to ce5fe81 Compare August 6, 2025 19:49
@vmaerten vmaerten force-pushed the fix/publish-nightly branch from bd068b1 to baef05d Compare August 6, 2025 20:00
@vmaerten vmaerten force-pushed the fix/publish-nightly branch from baef05d to c6f30e8 Compare August 6, 2025 20:02
Copy link
Member

@andreynering andreynering left a comment

Choose a reason for hiding this comment

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

Thanks for setting this up!

@vmaerten vmaerten merged commit 4bdfe5c into main Aug 9, 2025
15 checks passed
@vmaerten vmaerten deleted the fix/publish-nightly branch August 9, 2025 14:03
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.

3 participants