Skip to content

Migrate CLI module to match Fyne CLI tools - #355

Open
sdassow wants to merge 24 commits into
fyne-io:masterfrom
sdassow:migrate/cli
Open

Migrate CLI module to match Fyne CLI tools#355
sdassow wants to merge 24 commits into
fyne-io:masterfrom
sdassow:migrate/cli

Conversation

@sdassow

@sdassow sdassow commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description:

As a first step towards merging with fyne tools, only replace the CLI module, and should not change behaviour otherwise.

Checklist:

  • Tests included.
  • Lint and formatter run with no errors.
  • Tests all pass.

Where applicable:

  • Public APIs match existing style.
  • Any breaking changes have a deprecation path or have been discussed.
  • Updated the vendor folder (using go mod vendor).

@andydotxyz andydotxyz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This appears to be a breaking change - any parameters passed with the single-dash prefix will no longer work as it's moved to two. Can we support both to avoid this break?
[✗] invalid value "test" for flag -app-build: parse error

@sdassow

sdassow commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

This appears to be a breaking change - any parameters passed with the single-dash prefix will no longer work as it's moved to two. Can we support both to avoid this break? [✗] invalid value "test" for flag -app-build: parse error

Hm, are you sure? The -app-build in your example fails because it's expecting a number.
It works for me as expected with single and double dash options.

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