Skip to content

Add DryRun input to build-release workflow - #12

Open
TingluoHuang wants to merge 1 commit into
mainfrom
TingluoHuang-patch-1
Open

Add DryRun input to build-release workflow#12
TingluoHuang wants to merge 1 commit into
mainfrom
TingluoHuang-patch-1

Conversation

@TingluoHuang

Copy link
Copy Markdown
Member

Added DryRun input to control build/test behavior.

Added DryRun input to control build/test behavior.
CopilotAI lite review requested due to automatic review settings August 10, 2026 13:59
@TingluoHuang
TingluoHuang requested a review from a team as a code ownerAugust 10, 2026 13:59

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a DryRun input to the build/release workflow to allow skipping the release publication step while still building and testing the Alpine Node.js artifact.

Changes:

  • Introduced a boolean DryRun input for manual (workflow_dispatch) runs.
  • Gated the release job so it won’t run when DryRun is enabled.
Show a summary per file
FileDescription
.github/workflows/build-release-alpine-nodejs.ymlAdds a DryRun workflow input and uses it to conditionally skip the release job.

Review details

Tip

Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment on lines +11 to 15
DryRun:
description: 'DryRun will only build/test alpine node.js'
type: boolean
default: false
workflow_call:

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

this is expected.

Sign up for freeto 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

@TingluoHuang