-
-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release v1.16.0 #370
Release v1.16.0 #370
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #370 +/- ##
=======================================
Coverage 87.96% 87.96%
=======================================
Files 64 64
Lines 10433 10433
=======================================
Hits 9177 9177
Misses 1256 1256 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Just left you a change suggestion for CHANGELOG.md
to indicate the breaking changes.
packages/fleather/CHANGELOG.md
Outdated
* Enable auto-correct on mobile devices | ||
* Enable suggestions on Android devices | ||
* Adjust theme to dark mode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Enable auto-correct on mobile devices | |
* Enable suggestions on Android devices | |
* Adjust theme to dark mode | |
* [Behavior change] Add `autocorrect` which defaults to `true` | |
* [Behavior change] Add `enableSuggestions` which defaults to `true` | |
* [UI change] Adjust default theme |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should add to many tags to the release note for readability
It is not a breaking change it is an enhancement
Nevertheless, we must precise that auto correct only works for mobile and suggestions only on Android
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still prefer to let the user know it's going to break the UI and behavior but your version is fine too. Just please change the theme changelog since it affects the light theme too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks.
No description provided.