Skip to content

Tweak Apache Commons CLI call to work around a change there. - #3735

Open
cpovirk wants to merge 1 commit into
dart-lang:mainfrom
cpovirk:cliup
Open

cpovirk wants to merge 1 commit into
dart-lang:mainfrom
cpovirk:cliup

Conversation

@cpovirk

@cpovirk cpovirk commented Oct 9, 2026

Copy link
Copy Markdown

Calls to getOptionValue(string, null) have become ambiguous between
the long-standing (String, String)
overload

and the more recently introduced (String, Supplier)
overload
.

Description

Replace this paragraph with a description of what this PR is changing or adding, and why. The 'why' is the most important part of the description.

Related Issues

List which issues are fixed by this PR. Use the syntax Fixes #1234.

PR Checklist

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
  • I've run dart tool/ci.dart --all locally and resolved all issues identified. This ensures the PR is formatted, has no lint errors, and ran all code generators. This applies to the packages part of the toplevel pubspec.yaml workspace.
  • All existing and new tests are passing. I added new tests to check the change I am making.
  • The PR is actually solving the issue. PRs that don't solve the issue will be closed. Please be respectful of the maintainers' time. If it's not clear what the issue is, feel free to ask questions on the GitHub issue before submitting a PR.
  • I have updated CHANGELOG.md for the relevant packages. (Not needed for small changes such as doc typos).
  • I have updated the pubspec package version if necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant