feat: add 'create' subcommand to scaffold helmfile deployment projects by yxxhero · Pull Request #2574 · helmfile/helmfile

yxxhero · 2026-05-02T10:46:46Z

Summary

Add helmfile create [NAME] subcommand that generates a best-practice helmfile project scaffold
Generates helmfile.yaml (with commented examples for helmDefaults, repositories, environments, releases), environments/default.yaml, and values/.gitkeep
Supports --output-dir/-o for custom output path, --force to overwrite existing files, and validates project name against path traversal

Usage

helmfile create myproject          # creates ./myproject/ with scaffold
helmfile create                     # creates scaffold in current directory
helmfile create -o /custom/path    # creates scaffold in custom path
helmfile create myproject --force  # overwrite existing helmfile.yaml

Generated structure

myproject/
├── helmfile.yaml           # Main config with commented examples
├── environments/
│   └── default.yaml        # Default environment values
└── values/
    └── .gitkeep            # Placeholder for release values

Test plan

helmfile create myproject creates correct directory structure
helmfile create creates scaffold in current directory
--output-dir flag overrides output path
Duplicate detection refuses without --force, allows with --force
Invalid names with path separators (../, /) are rejected
go vet passes
go build succeeds

Add 'helmfile create [NAME]' command that generates a best-practice helmfile project structure with: - helmfile.yaml with commented examples (helmDefaults, repositories, environments, releases) - environments/default.yaml for environment-specific values - values/.gitkeep placeholder for release values Supports --output-dir/-o for custom output path and --force to overwrite existing files. Validates project name to prevent path traversal. Signed-off-by: yxxhero <[email protected]>

Copilot

Pull request overview

Adds a new helmfile create CLI subcommand to scaffold a “best-practice” Helmfile project directory, integrating into the existing Cobra command tree and app/config layers.

Changes:

Introduces helmfile create [NAME] command with --output-dir/-o and --force.
Adds config validation for project name and basic overwrite protection.
Implements scaffold generation (helmfile.yaml, environments/default.yaml, values/.gitkeep) in pkg/app.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
pkg/config/create.go	Adds create command options + validation (name rules, output dir resolution, basic overwrite check).
pkg/app/create.go	Implements scaffold directory/file creation and user-facing log output.
pkg/app/config.go	Adds `CreateConfigProvider` interface for app layer.
cmd/root.go	Registers the new `create` subcommand on the root command.
cmd/create.go	Wires Cobra command/flags to config validation and `App.Create`.

+	envDir := filepath.Join(absDir, "environments")
+	if err := os.MkdirAll(envDir, 0o755); err != nil {
+		return fmt.Errorf("failed to create directory %s: %w", envDir, err)
+	}
+
+	envFilePath := filepath.Join(envDir, "default.yaml")
+	if err := os.WriteFile(envFilePath, []byte(envDefaultYAMLTemplate), 0o644); err != nil {
+		return fmt.Errorf("failed to write %s: %w", envFilePath, err)
+	}
+	c.Logger().Infof("created %s", envFilePath)
+
+	valuesDir := filepath.Join(absDir, "values")
+	if err := os.MkdirAll(valuesDir, 0o755); err != nil {
+		return fmt.Errorf("failed to create directory %s: %w", valuesDir, err)
+	}
+
+	gitkeepPath := filepath.Join(valuesDir, ".gitkeep")
+	if err := os.WriteFile(gitkeepPath, []byte(""), 0o644); err != nil {
+		return fmt.Errorf("failed to write %s: %w", gitkeepPath, err)
+	}


+	helmfilePath := filepath.Join(absDir, "helmfile.yaml")
+	if !c.Force() {
+		if _, err := os.Stat(helmfilePath); err == nil {
+			return fmt.Errorf("helmfile.yaml already exists in %s, use --force to overwrite", absDir)
+		}


+func (a *App) Create(c CreateConfigProvider) error {
+	outputDir := c.OutputDir()
+	absDir, err := filepath.Abs(outputDir)
+	if err != nil {
+		return fmt.Errorf("failed to resolve output directory: %w", err)
+	}
+
+	if err := os.MkdirAll(absDir, 0o755); err != nil {
+		return fmt.Errorf("failed to create directory %s: %w", absDir, err)


yxxhero · 2026-05-02T11:04:50Z

@copilot fix comments.

…or create command - pkg/app/create.go: extract writeFileIfNotExists helper that respects the --force flag; all three scaffold files (helmfile.yaml, environments/default.yaml, values/.gitkeep) now refuse to overwrite without --force - pkg/config/create.go: ValidateConfig now checks all three scaffold paths and reports every already-existing file in a single error before proceeding, instead of only checking helmfile.yaml - pkg/app/create_test.go: add unit tests covering new directory, current directory, per-file overwrite rejection without --force, and full overwrite with --force Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/eb6d9e4b-0f72-4e26-b841-e1e39a2b2e83 Co-authored-by: yxxhero <[email protected]>

Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/eb6d9e4b-0f72-4e26-b841-e1e39a2b2e83 Co-authored-by: yxxhero <[email protected]>

Copilot · 2026-05-02T11:14:55Z

@copilot fix comments.

Fixed in commits 0689726 and afffa0e:

pkg/app/create.go: Extracted a writeFileIfNotExists helper — when --force is false it checks each file before writing and returns an error instead of overwriting. All three scaffold files (helmfile.yaml, environments/default.yaml, values/.gitkeep) are now guarded individually.
pkg/config/create.go: ValidateConfig now checks all three scaffold paths and reports every already-existing file in a single error (rather than only checking helmfile.yaml).
pkg/app/create_test.go: Added unit tests covering: new directory scaffold, current-directory scaffold, per-file overwrite rejection without --force (one test per file), and full overwrite with --force.

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

+func (c *CreateImpl) ValidateConfig() error {
+	name := c.CreateOptions.Name
+	if name != "" {
+		if strings.ContainsAny(name, "/\\") {
+			return fmt.Errorf("invalid project name %q: must not contain path separators", name)
+		}
+		if name == ".." {
+			return fmt.Errorf("invalid project name %q", name)
+		}
+		if strings.TrimSpace(name) == "" {
+			return fmt.Errorf("project name must not be empty or whitespace only")
+		}
+	}


+	f.StringVarP(&options.OutputDir, "output-dir", "o", "", "Output directory (default: NAME or current directory)")
+	f.BoolVar(&options.Force, "force", false, "Overwrite existing helmfile.yaml")
+


+			createImpl := config.NewCreateImpl(globalCfg, options)
+			if err := createImpl.ValidateConfig(); err != nil {
+				return err
+			}
+			a := app.New(createImpl)
+			return toCLIError(createImpl.GlobalImpl, a.Create(createImpl))


+			return fmt.Errorf("the following files already exist, use --force to overwrite: %s", strings.Join(existing, ", "))
+		}
+	}
+	return nil


yxxhero · 2026-05-02T11:17:21Z

@copilot fix comments.

- cmd/create.go: add config.NewCLIConfigImpl() call for consistency with other subcommands; update --force flag help text to list all overwritten files - pkg/config/create.go: delegate to c.GlobalImpl.ValidateConfig() at end of ValidateConfig() for global option validation (--color/--no-color) - pkg/config/create_test.go: add unit tests for CreateImpl.ValidateConfig() covering path separator rejection, '..' rejection, existing-file detection per-file and with --force, and global color conflict delegation Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/6327d657-5888-4b94-85fb-def80c0a193f Co-authored-by: yxxhero <[email protected]>

Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/6327d657-5888-4b94-85fb-def80c0a193f Co-authored-by: yxxhero <[email protected]>

Copilot · 2026-05-02T11:27:41Z

@copilot fix comments.

Fixed in commit e7e6569 (with a follow-up cleanup in 6f56010):

cmd/create.go: Added config.NewCLIConfigImpl() call before ValidateConfig() (consistent with init, apply, etc.); updated --force help text to list all three overwritten files.
pkg/config/create.go: ValidateConfig() now delegates to c.GlobalImpl.ValidateConfig() at the end so global option validation (e.g. --color + --no-color conflict) is enforced.
pkg/config/create_test.go (new): Unit tests for CreateImpl.ValidateConfig() covering path separator rejection, .. rejection, per-file existing-file detection without --force, --force bypass, and global color conflict delegation.

Copilot

Pull request overview

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

+	helmfilePath := filepath.Join(absDir, "helmfile.yaml")
+	if err := writeFileIfNotExists(helmfilePath, []byte(helmfileYAMLTemplate), c.Force()); err != nil {
+		return err
+	}
+	c.Logger().Infof("created %s", helmfilePath)


yxxhero · 2026-05-02T12:19:38Z

@copilot fix comments.

Refactor Create to collect all conflicting scaffold paths up front before writing anything. When --force is not set and any scaffold file already exists, the command returns a single error listing all conflicts without touching the filesystem. Also removes the now-unnecessary writeFileIfNotExists helper and adds a test (TestCreate_PreflightAtomicOnLaterConflict) verifying that a conflict on a later file (e.g. environments/default.yaml) prevents even the first file (helmfile.yaml) from being created. Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/aae6f2e6-7f9e-42b8-afa3-78edd3215127 Co-authored-by: yxxhero <[email protected]>

yxxhero · 2026-05-02T23:43:40Z

@copilot fix comments.

…or TOCTOU protection - pkg/config/create.go: remove file-existence check from ValidateConfig (duplicate of App.Create's preflight); ValidateConfig now only validates the project name and delegates to GlobalImpl.ValidateConfig. Remove unused os/path/filepath imports. - pkg/app/create.go: add writeScaffoldFile helper that uses O_CREATE|O_EXCL when force=false, so a file appearing between the preflight check and the actual write is caught rather than silently overwritten (TOCTOU protection). - pkg/config/create_test.go: remove four file-existence tests that tested the now-deleted ValidateConfig logic; file-conflict coverage remains in pkg/app/create_test.go. Simplify ValidName and GlobalColorConflict tests. Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/82f82e72-934f-416c-8662-5060e92284fa Co-authored-by: yxxhero <[email protected]>

…tests - pkg/app/create.go: wrap os.IsExist error from writeScaffoldFile with a message that names the conflicting file and suggests --force, so the user gets actionable output even in the TOCTOU case - pkg/app/create_test.go: add TestWriteScaffoldFile_CreatesNewFile, TestWriteScaffoldFile_ExistingFileNoForce, and TestWriteScaffoldFile_ExistingFileWithForce to cover the helper directly Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/82f82e72-934f-416c-8662-5060e92284fa Co-authored-by: yxxhero <[email protected]>

Copilot · 2026-05-02T23:55:38Z

@copilot fix comments.

Fixed in commits 542bf28 and 91898f8:

pkg/config/create.go: Removed the file-existence check from ValidateConfig entirely — filesystem checks are now delegated to App.Create only (one layer), eliminating the duplication. ValidateConfig now only validates the project name and delegates to GlobalImpl.ValidateConfig().
pkg/app/create.go: Added writeScaffoldFile helper that uses O_CREATE|O_EXCL when force=false (TOCTOU protection). The os.IsExist error from O_EXCL is wrapped with a message naming the conflicting file and suggesting --force.
pkg/app/create_test.go: Added three direct tests for writeScaffoldFile (new file, existing file without force, existing file with force).
pkg/config/create_test.go: Removed the four file-existence tests that tested the now-deleted ValidateConfig logic (conflict coverage remains in pkg/app/create_test.go).

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

+func (a *App) Create(c CreateConfigProvider) error {
+	outputDir := c.OutputDir()
+	absDir, err := filepath.Abs(outputDir)
+	if err != nil {
+		return fmt.Errorf("failed to resolve output directory: %w", err)
+	}


+		if strings.ContainsAny(name, "/\\") {
+			return fmt.Errorf("invalid project name %q: must not contain path separators", name)
+		}
+		if name == ".." {
+			return fmt.Errorf("invalid project name %q", name)
+		}
+		if strings.TrimSpace(name) == "" {


+func TestCreateImpl_ValidateConfig_ValidName(t *testing.T) {
+	c := newTestCreateImplWithDefaults("myproject", "", false)
+	require.NoError(t, c.ValidateConfig())
+}


yxxhero · 2026-05-03T01:27:08Z

@copilot fix comments

yxxhero · 2026-05-03T01:27:31Z

@copilot fix ci issues

…; add '.' name test - pkg/app/create.go: wrap all App.Create fmt.Errorf returns with appError("", ...) so toCLIError produces a clean user-friendly message instead of "unexpected error: *fmt.wrapError: ..." - pkg/config/create.go: reject "." as a NAME alongside ".." to prevent accidentally scaffolding into the current directory via a named argument - pkg/config/create_test.go: add TestCreateImpl_ValidateConfig_NameDot Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/6d64508e-2d66-47e9-a02a-7669a2f481b7 Co-authored-by: yxxhero <[email protected]>

Copilot · 2026-05-03T01:34:12Z

@copilot fix comments

Fixed in commit ee1fda3:

pkg/app/create.go: All fmt.Errorf returns are now wrapped with appError("", ...) so toCLIError routes them through the *app.Error case instead of the default case that produces "unexpected error: *fmt.wrapError: ...".
pkg/config/create.go: Added "." to the sentinel check alongside ".." so helmfile create . is rejected with invalid project name ".".
pkg/config/create_test.go: Added TestCreateImpl_ValidateConfig_NameDot covering the new rejection.

… error Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/11cd65e9-c5ef-4195-9375-bc929169616b Co-authored-by: yxxhero <[email protected]>

Copilot · 2026-05-03T01:39:13Z

@copilot fix ci issues

Fixed in commit fcce6f9. The lint failure was unparam flagging the outputDir parameter in newTestCreateImplWithDefaults because every caller passed "". Removed the parameter and dropped OutputDir from the helper — all call sites updated accordingly.

yxxhero · 2026-05-03T08:52:05Z

@copilot fix ci issues

Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/0e1bdac5-708f-4615-ae6d-e22fc1e921f2 Co-authored-by: yxxhero <[email protected]>

Copilot · 2026-05-03T08:57:21Z

@copilot fix ci issues

Fixed in commit e736132. The unparam linter flagged the force parameter in newTestCreateImplWithDefaults because every caller passed false. Removed the parameter (and Force field from the helper's CreateOptions) — all call sites updated accordingly.

#2574) * feat: add 'create' subcommand to scaffold helmfile deployment projects Add 'helmfile create [NAME]' command that generates a best-practice helmfile project structure with: - helmfile.yaml with commented examples (helmDefaults, repositories, environments, releases) - environments/default.yaml for environment-specific values - values/.gitkeep placeholder for release values Supports --output-dir/-o for custom output path and --force to overwrite existing files. Validates project name to prevent path traversal. Signed-off-by: yxxhero <[email protected]> * fix: add overwrite protection for all scaffold files and unit tests for create command - pkg/app/create.go: extract writeFileIfNotExists helper that respects the --force flag; all three scaffold files (helmfile.yaml, environments/default.yaml, values/.gitkeep) now refuse to overwrite without --force - pkg/config/create.go: ValidateConfig now checks all three scaffold paths and reports every already-existing file in a single error before proceeding, instead of only checking helmfile.yaml - pkg/app/create_test.go: add unit tests covering new directory, current directory, per-file overwrite rejection without --force, and full overwrite with --force Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/eb6d9e4b-0f72-4e26-b841-e1e39a2b2e83 Co-authored-by: yxxhero <[email protected]> * fix: remove redundant absDir from ValidateConfig error message Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/eb6d9e4b-0f72-4e26-b841-e1e39a2b2e83 Co-authored-by: yxxhero <[email protected]> * fix: address create command review feedback - cmd/create.go: add config.NewCLIConfigImpl() call for consistency with other subcommands; update --force flag help text to list all overwritten files - pkg/config/create.go: delegate to c.GlobalImpl.ValidateConfig() at end of ValidateConfig() for global option validation (--color/--no-color) - pkg/config/create_test.go: add unit tests for CreateImpl.ValidateConfig() covering path separator rejection, '..' rejection, existing-file detection per-file and with --force, and global color conflict delegation Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/6327d657-5888-4b94-85fb-def80c0a193f Co-authored-by: yxxhero <[email protected]> * fix: clarify test helper name and comment in create_test.go Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/6327d657-5888-4b94-85fb-def80c0a193f Co-authored-by: yxxhero <[email protected]> * fix: atomic preflight check in App.Create before any writes Refactor Create to collect all conflicting scaffold paths up front before writing anything. When --force is not set and any scaffold file already exists, the command returns a single error listing all conflicts without touching the filesystem. Also removes the now-unnecessary writeFileIfNotExists helper and adds a test (TestCreate_PreflightAtomicOnLaterConflict) verifying that a conflict on a later file (e.g. environments/default.yaml) prevents even the first file (helmfile.yaml) from being created. Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/aae6f2e6-7f9e-42b8-afa3-78edd3215127 Co-authored-by: yxxhero <[email protected]> * fix: handle non-IsNotExist Stat errors in preflight check; add whitespace name test; fix gci formatting - pkg/app/create.go: treat os.Stat errors that are NOT os.IsNotExist as hard errors in the preflight scan, surfacing permission/IO issues before any writes happen; remove trailing blank line that caused gci failure - pkg/config/create.go: same non-IsNotExist error handling in ValidateConfig - pkg/config/create_test.go: add TestCreateImpl_ValidateConfig_WhitespaceOnlyName covering the " " (whitespace-only) name rejection branch Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/d6574f56-f46d-46f7-99d9-e0b0b897b3b5 Co-authored-by: yxxhero <[email protected]> * refactor: eliminate duplicated scaffold existence check; use O_EXCL for TOCTOU protection - pkg/config/create.go: remove file-existence check from ValidateConfig (duplicate of App.Create's preflight); ValidateConfig now only validates the project name and delegates to GlobalImpl.ValidateConfig. Remove unused os/path/filepath imports. - pkg/app/create.go: add writeScaffoldFile helper that uses O_CREATE|O_EXCL when force=false, so a file appearing between the preflight check and the actual write is caught rather than silently overwritten (TOCTOU protection). - pkg/config/create_test.go: remove four file-existence tests that tested the now-deleted ValidateConfig logic; file-conflict coverage remains in pkg/app/create_test.go. Simplify ValidName and GlobalColorConflict tests. Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/82f82e72-934f-416c-8662-5060e92284fa Co-authored-by: yxxhero <[email protected]> * fix: wrap O_EXCL error with --force hint; add writeScaffoldFile unit tests - pkg/app/create.go: wrap os.IsExist error from writeScaffoldFile with a message that names the conflicting file and suggests --force, so the user gets actionable output even in the TOCTOU case - pkg/app/create_test.go: add TestWriteScaffoldFile_CreatesNewFile, TestWriteScaffoldFile_ExistingFileNoForce, and TestWriteScaffoldFile_ExistingFileWithForce to cover the helper directly Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/82f82e72-934f-416c-8662-5060e92284fa Co-authored-by: yxxhero <[email protected]> * fix: wrap App.Create errors in *app.Error; reject '.' as project name; add '.' name test - pkg/app/create.go: wrap all App.Create fmt.Errorf returns with appError("", ...) so toCLIError produces a clean user-friendly message instead of "unexpected error: *fmt.wrapError: ..." - pkg/config/create.go: reject "." as a NAME alongside ".." to prevent accidentally scaffolding into the current directory via a named argument - pkg/config/create_test.go: add TestCreateImpl_ValidateConfig_NameDot Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/6d64508e-2d66-47e9-a02a-7669a2f481b7 Co-authored-by: yxxhero <[email protected]> * fix: drop unused outputDir param from test helper to fix unparam lint error Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/11cd65e9-c5ef-4195-9375-bc929169616b Co-authored-by: yxxhero <[email protected]> * fix: drop unused force param from test helper to fix unparam lint error Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/0e1bdac5-708f-4615-ae6d-e22fc1e921f2 Co-authored-by: yxxhero <[email protected]> --------- Signed-off-by: yxxhero <[email protected]> Co-authored-by: copilot-swe-agent[bot] <[email protected]>

yxxhero requested review from aditmeno, itscaro, mumoshu, xiaomudk and zhaque44 as code owners May 2, 2026 10:46

yxxhero requested a review from Copilot May 2, 2026 10:53

Copilot started reviewing on behalf of yxxhero May 2, 2026 10:53 View session

Copilot AI reviewed May 2, 2026

View reviewed changes

Copilot started work on behalf of yxxhero May 2, 2026 11:04 View session

yxxhero requested a review from Copilot May 2, 2026 11:12

Copilot started reviewing on behalf of yxxhero May 2, 2026 11:13 View session

fix: remove redundant absDir from ValidateConfig error message

afffa0e

Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/eb6d9e4b-0f72-4e26-b841-e1e39a2b2e83 Co-authored-by: yxxhero <[email protected]>

Copilot finished work on behalf of yxxhero May 2, 2026 11:15

Copilot AI reviewed May 2, 2026

View reviewed changes

Copilot started work on behalf of yxxhero May 2, 2026 11:17 View session

Copilot AI and others added 2 commits May 2, 2026 11:23

fix: clarify test helper name and comment in create_test.go

6f56010

Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/6327d657-5888-4b94-85fb-def80c0a193f Co-authored-by: yxxhero <[email protected]>

Copilot finished work on behalf of yxxhero May 2, 2026 11:28

yxxhero requested a review from Copilot May 2, 2026 11:51

Copilot started reviewing on behalf of yxxhero May 2, 2026 11:51 View session

Copilot AI reviewed May 2, 2026

View reviewed changes

Comment thread pkg/app/create.go Outdated

Comment on lines +78 to +82

helmfilePath := filepath.Join(absDir, "helmfile.yaml")

if err := writeFileIfNotExists(helmfilePath, []byte(helmfileYAMLTemplate), c.Force()); err != nil {

return err

}

c.Logger().Infof("created %s", helmfilePath)

Copilot started work on behalf of yxxhero May 2, 2026 12:19 View session

Copilot started work on behalf of yxxhero May 2, 2026 23:43 View session

Copilot AI and others added 2 commits May 2, 2026 23:51

Copilot finished work on behalf of yxxhero May 2, 2026 23:56

yxxhero requested a review from Copilot May 3, 2026 00:00

Copilot started reviewing on behalf of yxxhero May 3, 2026 00:01 View session

Copilot AI reviewed May 3, 2026

View reviewed changes

Copilot started work on behalf of yxxhero May 3, 2026 01:27 View session

Copilot finished work on behalf of yxxhero May 3, 2026 01:34

Copilot started work on behalf of yxxhero May 3, 2026 01:34 View session

Copilot finished work on behalf of yxxhero May 3, 2026 01:39

Copilot started work on behalf of yxxhero May 3, 2026 08:52 View session

fix: drop unused force param from test helper to fix unparam lint error

e736132

Agent-Logs-Url: https://github.com/helmfile/helmfile/sessions/0e1bdac5-708f-4615-ae6d-e22fc1e921f2 Co-authored-by: yxxhero <[email protected]>

Copilot finished work on behalf of yxxhero May 3, 2026 08:57

xiaomudk approved these changes May 3, 2026

View reviewed changes

yxxhero merged commit 902c5ce into main May 3, 2026
16 checks passed

yxxhero deleted the feat/create-command branch May 3, 2026 11:03

		f.StringVarP(&options.OutputDir, "output-dir", "o", "", "Output directory (default: NAME or current directory)")
		f.BoolVar(&options.Force, "force", false, "Overwrite existing helmfile.yaml")

Uh oh!

Uh oh!

Conversation

yxxhero commented May 2, 2026

Summary

Usage

Generated structure

Test plan

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

yxxhero commented May 2, 2026

Uh oh!

Copilot AI commented May 2, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

yxxhero commented May 2, 2026

Uh oh!

Copilot AI commented May 2, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

yxxhero commented May 2, 2026

Uh oh!

yxxhero commented May 2, 2026

Uh oh!

Copilot AI commented May 2, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

yxxhero commented May 3, 2026

Uh oh!

yxxhero commented May 3, 2026

Uh oh!

Copilot AI commented May 3, 2026

Uh oh!

Copilot AI commented May 3, 2026

Uh oh!

yxxhero commented May 3, 2026

Uh oh!

Copilot AI commented May 3, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants