Prepare reel_text 0.4.2 reliability fixes - #29
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 183c6e9828
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
reel_text0.4.2TextPainterinstances on success and failure pathsReelTextControllerinstancesgoogle_fonts8.2.0)Why
The measurement path retained its temporary
TextPainter, controller replacement could leave a target queued by the previous controller, and invalid timing values were accepted until lower-level Flutter APIs failed. These changes make those ownership and transition boundaries explicit while preserving the current public API.Impact
Invalid timing input now fails consistently with
ArgumentErrorat the API consumption boundary. Valid animations and the visual rendering model are unchanged.Release
pubspec.yamland the example lockfile to 0.4.2mainafter merging Dependabot PR Bump google_fonts from 8.1.0 to 8.2.0 in /example in the example-dependencies group #30Validation
dart format --output=none --set-exit-if-changed lib testflutter analyzeflutter test— 101 testsflutter test --platform chrome— 101 testscd example && flutter analyzecd example && flutter test— 28 testsgit diff --checkdart pub publish --dry-run— 0 warningsflutter build macos --debugReadytoInvite copiedin active and settled states