release: 1.5.1 - #198
Conversation
Two fixes, both in `WText`'s casing transforms and both found by driving a real app rather than by reading the code. `uppercase` / `lowercase` / `capitalize` now cast under the ambient locale, so a Turkish app renders `İZLEYİCİLER` and `GÜVENLİK` instead of `IZLEYICILER` and `GÜVENLIK` (#195). And `capitalize` raises the first letter of every word rather than of the string, which is what its own documentation, the demo gallery and CSS have promised all along (#197). The word rule was measured in Chromium over 32 strings, so `well-known` is `Well-Known`, `3rd party` stays `3rd Party`, and decomposed NFD text no longer capitalises mid-word. Expect one visible change on upgrade: a multi-word string carrying `capitalize` now capitalises every word. Five surfaces bumped, the patch-release set: `pubspec.yaml`, `example/pubspec.yaml`, the `dartdoc_options.yaml` source-link tag, the `llms.txt` version string, and the `CHANGELOG.md` promotion with its two link references. `skills/wind-ui/` needs no version move for a patch: the H1, the description prefix and the `1.5.x` marker all still read right, and the skill's own version went to 2.13.2 with the content change in #197. Gates: `dart analyze` clean, `dart format` no diff, `flutter test` 1760 passing with the one pre-existing skip, `./tool/coverage.sh 90` at 94.8%, `tool/check-docs.py` 0 issues, and `dart pub publish --dry-run` clean once this commit lands (the only warning it raised was this bump sitting uncommitted).
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe project version was updated from 1.5.0 to 1.5.1 across package metadata, the example application, documentation, dartdoc links, and changelog references. ChangesRelease metadata
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to The release changelog and version references are aligned for 1.5.1, with no remaining merge-readiness risk identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Note Kodizm (AI-generated). May contain mistakes; verify before acting. Clean release bump: all five version surfaces are consistent at 1.5.1, every gate passes locally, and the one gap is a lockfile the previous release chose to commit. Minor
Not from this PR, worth a glance before tagging
Version surfacesChecked each against the base: TestsNo source change here, so nothing new to cover. The two fixes this release ships landed on Checks I ran
|
`example/pubspec.lock` records the path dependency's version, so it still read 1.5.0 and the first `flutter pub get` in `example/` after this merge would have left a dirty tree. The 1.5.0 release committed this same line for the same reason and its message says so; 1.4.1 omitted it, which is why the entry was stale going into that release. Produced by running `flutter pub get` in `example/` rather than by hand, and the diff is the one line: no `source: path` churn from the gitignored `pubspec_overrides.yaml` this time, and the root `pubspec.lock` is untouched.
|
Taken in fc0c12e, and the precedent is exactly as described: Produced by running So the patch-release surface is seven files, not the six the checklist has been carrying. Noted for the next one. The second note is right too and stays out of this PR: the root |
|
Note Kodizm (AI-generated). May contain mistakes; verify before acting. Incremental re-review of the three commits since What changed: No findings. TestsNo source change in these commits; the CI workflow edit is not unit-testable and #199 carried its own measured verification against pinned lychee 0.24.2. Checks I ran
|
What
Patch release. Two fixes, both in
WText's casing transforms.uppercase/lowercase/capitalizecast under the ambient locale, so a Turkish app rendersİZLEYİCİLERandGÜVENLİKinstead ofIZLEYICILERandGÜVENLIK. With noLocalizationsancestor the transform falls back to Dart's locale-independent casing, which is what every bare widget test gets.capitalize(fix(w_text): capitalize every word, not just the leading letter #197). It raised the first letter of the string, where the doc page, the demo gallery and CSS all promise the first letter of every word. The word rule was measured in Chromium over 32 strings:well-knownisWell-Known,read/writeisRead/Write, while3rd partystays3rd Party,l'orangestaysL'orangeand decomposed NFD text no longer capitalises mid-word.Expect on upgrade: a multi-word string carrying
capitalizenow capitalises every word, where before only the leading letter moved.Version surfaces
The patch set, five files:
pubspec.yamlversion: 1.5.1example/pubspec.yamlversion: 1.5.1+1dartdoc_options.yamlblob/1.5.1/llms.txtVersion 1.5.1 stableCHANGELOG.md[Unreleased]promoted to[1.5.1] - 2026-09-07, tag reference added, compare reference retargetedskills/wind-ui/needs no version move for a patch: the H1, thedescriptionprefix and thefluttersdk_wind 1.5.xmarker all still read right, and the skill's own version went to 2.13.2 with the content change in #197.Testing
dart analyze: clean in the package and inexample/dart format --set-exit-if-changed: no diffflutter test: 1760 passing, the one pre-existing skip./tool/coverage.sh 90: 94.8%python3 tool/check-docs.py: 0 issuesdart pub publish --dry-run: the only warning was this bump sitting uncommitted, so it clears with the commitawkover the## [1.5.1]section, the check that fails thegithub-releasejob on empty notes): returns both entriesAfter merge
git tag 1.5.1 && git push origin 1.5.1triggerspublish.yml: validate, pub.dev publish over OIDC, the GitHub Release, and the registry sync job that pushesskills/wind-ui/tofluttersdk/ai. pub.dev cannot unpublish, only retract, so the tag push is the point of no return.Summary by CodeRabbit