-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[update] local_auth - intl version #2084
Conversation
|
Do you know whether the breaking change to the intl package is breaking for the users of local_auth? (if so this should be a major bump for local_auth as well). |
|
I just bumped it for a need in my app, and I required no changes! |
We should understand whether this is the case for all use cases. |
|
Ok what can we do to test that? |
|
We should understand what is the breaking change, and whether/how it affect this plugin. |
|
The Flutter on the master channel localizations uses intl 0.16.0 and so do many plugins, so regardless, having a version that supports the latest can be an option for those requiring it. As for upgrade path, they can continue to use 0.6.0 since there are no changes. |
|
Agreed that we need to support intl 0.16.0, I want to figure versioning correctly though, if we do an unnecessary major version bump it means that future bugfixes may not arrive to all apps that can get them. |
|
Gotcha, I understand. What about a minor version bump 0.6.1? |
|
That's why we need to understand if this is a breaking change for the users of this plugin, if it is then this must be a major version bump, otherwise this shouldn't be a major version bump. |
|
Ok sounds good. I made it a minor version bump. Unless we find a breaking change. |
|
What we should do is understand the nature of the breaking change to intl and whether/how it affects the users of this plugin. |
|
OK sounds good. Here are the changes to intl since 0.15.1 -> 0.16.0 0.16.0
0.15.8
0.15.7
0.15.6
0.15.5
0.15.4
0.15.3
0.15.2
|
|
looks like it was mostly bug fixes but |
|
It's not completely clear from the changelog what is the breaking change in 0.16.0, filed dart-lang/i18n#169 to ask for a clarification. |
|
Seems like the major version bump for the intl package was a mistake, we should use |
|
Ok sounds great 👍🏼 |
|
OK i just updated it to use |
packages/local_auth/pubspec.yaml
Outdated
| author: Flutter Team <[email protected]> | ||
| homepage: https://github.com/flutter/plugins/tree/master/packages/local_auth | ||
| version: 0.6.0 | ||
| version: 0.6.1 |
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.
Should be 0.6.0+1
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 pushed an update
amirh
left a comment
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
Upgrade local auth
intl-> ^0.16.0