Skip to content

[CP] Make gen-l10n error handling independent of logger state #119678

Description

@AlexV525

issue_link

#119593

Commit Hash

3884381

Target

stable

pr_link

#119679

Impacted Users

Users that are using any overridden FLUTTER_STORAGE_BASE_URL

Impact Description

The issue is breaking every user that is using localizations and a custom FLUTTER_STORAGE_BASE_URL. The gen-l10n tool will report Found syntax errors but not any actual syntax errors.

Workaround

#119593 (comment) or CP #119644.

Risk

low

Test Coverage

yes

Validation Steps

Follow the steps to reproduce in #119593 (comment).

Activity

  1. AlexV525 commented on Feb 2, 2023

    @AlexV525
    MemberAuthor
  2. christopherfujino commented on Feb 2, 2023

    @christopherfujino
    Contributor

    This CP LGTM

  3. github-actions commented on Mar 3, 2023

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  4. locked as resolved and limited conversation to collaborators on Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

cp: approvedApproved cherry-pick requestcp: merge-to-stableCherry-picks that should be merged to stablecp: reviewCherry-picks in the review queue

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions