Skip to content

Conversation

@thkim1011
Copy link
Contributor

This PR fixes ignoring when random positional arguments added to the flutter gen-l10n.

So we are no longer able to call flutter gen-l10n hello world or flutter gen-l10n --format false.

Fixes #118203

@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Jul 10, 2023
Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

@thkim1011 thkim1011 merged commit dabd7b3 into flutter:master Jul 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 12, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 12, 2023
gbtb16 pushed a commit to gbtb16/flutter that referenced this pull request Nov 6, 2023
This PR fixes ignoring when random positional arguments added to the
`flutter gen-l10n`.

So we are no longer able to call `flutter gen-l10n hello world` or
`flutter gen-l10n --format false`.

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

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

gen-l10n ignores explicit false values for arguments

2 participants