Skip to content

fix(promote): snapcraft cannot promote from edge with the --yes flag#27

Open
Guillaumebeuzeboc wants to merge 1 commit intomainfrom
fix/snapcraft-unwritten-rules
Open

fix(promote): snapcraft cannot promote from edge with the --yes flag#27
Guillaumebeuzeboc wants to merge 1 commit intomainfrom
fix/snapcraft-unwritten-rules

Conversation

@Guillaumebeuzeboc
Copy link
Collaborator

Since not being able to promote from edge with the --yes flag is by design we must trick snapcraft.

Copy link

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 addresses a limitation in snapcraft where promoting from the edge channel cannot be done non-interactively using the --yes flag. The solution pipes yes into the snapcraft promote command and sets the SNAPCRAFT_HAS_TTY environment variable to bypass this restriction.

Key Changes

  • Replaced --yes flag with yes | pipe to provide interactive confirmation
  • Added SNAPCRAFT_HAS_TTY: "true" environment variable to simulate TTY presence
  • Added detailed comments explaining the workaround and referencing the upstream issue

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

Copy link

@artivis artivis left a comment

Choose a reason for hiding this comment

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

What is we simply published on candidate instead?

@Guillaumebeuzeboc
Copy link
Collaborator Author

Guillaumebeuzeboc commented Jan 8, 2026

What is we simply published on candidate instead?

But someone might want to use this workflow to promote from edge to beta.

@artivis
Copy link

artivis commented Jan 8, 2026

Well we're not obliged to accommodate every use cases, especially if they deviate much from our opinionated/expected workflow.
Let's discuss that further.

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