Skip to content

Conversation

@Hook25
Copy link
Collaborator

@Hook25 Hook25 commented Mar 3, 2025

Description

This replaces the old condition (which doesn't work) with a new one, an input parameter. It is useful to have this workflow "triggerable" without pushing to the store so that is why I kept it.

Minor: Changed the order in the daily builds of the snap and deb build so that it is easier to read (snap first, debs then)

Resolved issues

Documentation

Tests

https://github.com/canonical/checkbox/actions/runs/13632603247

cancelled because it worked. See the workflow that was executed and didnt upload. I didnt want to waste the compute hours.

Daily edited to check that they will upload to the store:
https://github.com/canonical/checkbox/actions/runs/13633093858/job/38105819081#step:4:5

@Hook25 Hook25 force-pushed the remove_workflow_call branch 2 times, most recently from fa8a58e to 4de3662 Compare March 3, 2025 14:39
Copy link
Collaborator

@pieqq pieqq left a comment

Choose a reason for hiding this comment

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

LGTM

@Hook25 Hook25 merged commit 2b97197 into main Mar 3, 2025
13 of 34 checks passed
@Hook25 Hook25 deleted the remove_workflow_call branch March 3, 2025 14:50
stanley31huang pushed a commit that referenced this pull request Mar 28, 2025
Replace condition with input parameter
mreed8855 pushed a commit that referenced this pull request Jul 31, 2025
Replace condition with input parameter
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