Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Conversation

@rodydavis
Copy link
Contributor

Upgrade local auth intl -> ^0.16.0

@amirh
Copy link
Contributor

amirh commented Sep 24, 2019

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).

cc @collinjackson

@rodydavis
Copy link
Contributor Author

I just bumped it for a need in my app, and I required no changes!

@amirh
Copy link
Contributor

amirh commented Sep 24, 2019

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.

@rodydavis
Copy link
Contributor Author

Ok what can we do to test that?

@amirh
Copy link
Contributor

amirh commented Sep 24, 2019

We should understand what is the breaking change, and whether/how it affect this plugin.

@rodydavis
Copy link
Contributor Author

rodydavis commented Sep 24, 2019

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.

@amirh
Copy link
Contributor

amirh commented Sep 24, 2019

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.

@rodydavis
Copy link
Contributor Author

Gotcha, I understand. What about a minor version bump 0.6.1?

@amirh
Copy link
Contributor

amirh commented Sep 24, 2019

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.

@rodydavis
Copy link
Contributor Author

Ok sounds good. I made it a minor version bump. Unless we find a breaking change.

@amirh
Copy link
Contributor

amirh commented Sep 24, 2019

What we should do is understand the nature of the breaking change to intl and whether/how it affects the users of this plugin.

@rodydavis
Copy link
Contributor Author

OK sounds good. Here are the changes to intl since 0.15.1 -> 0.16.0

0.16.0

  • Fix 'k' formatting (1 to 24 hours) which incorrectly showed 0 to 23.
  • Tighten up types in a couple of places.
  • Add dart2js pragmas for inlining to help remove descriptions and other
    compile-time information from the output.

0.15.8

  • Add return type to some internal methods to improve dart2js output.
  • Change parameter types in some public methods from dynamic (implicit or
    explicit) to Object. In particular, the examples and args parameters on
    Intl.message, Intl.plural, Intl.gender, and Intl.select, as well as the args
    parameter on MessageLookup.
  • Allow Dart enums in an Intl.select call. The map of cases can either take
    enums directly, or the short string name of the enum. Requires
    intl_translation 0.17.4 to take advantage of this.

0.15.7

  • Upate to require Dart 2.0. Remove deprecated calls,
  • Compensate for rare cases where a parsed Date in date-only format gets a
    1:00am time. This is presumably because of DST time-shifts. We may not be
    able to correct these dates, because midnight may not exist at a transition
    date, but we can cause the strict parsing to not fail for these dates.
  • Update tests to split VM and web number tests, since larger integers now fail
    to compile with dart2js.

0.15.6

  • More upper case constant removal.

0.15.5

  • Add type parameters on numberFormatSymbols for Dart 2 compatibility. Note
    that it only adds them on the right-hand side because adding them to the
    static type can cause unnecessary cast warnings.
  • Replace uses of JSON constant for Dart 2 compatibility.

0.15.4

  • A couple of minor Dart 2 fixes.

0.15.3

  • Add a customPattern parameter to the currency constructor. This can be used
    to provide a custom pattern if you have one, e.g. for accounting formats.
  • Update data to CLDR 32.0.1
  • Update for Dart 2.0 fixed-size integers.
  • Add missing support for specifying decimalDigits in compactSimpleCurrency.
  • Fix doc comments for DateFormat (Pull request Fix doc comments for DateFormat. dart-archive/intl#156)
  • Added a skip argument to not output the message in the extract step.
  • Compensate for parsing a Date that happens at a DST transition, particularly
    in Brazil, where the transition happens at midnight. This can result in a
    time of 11:00pm the previous day, or of 1:00am the next day. Make sure that
    the the 11:00pm case does not cause us to get the wrong date.

0.15.2

  • Group the padding digits to the left of the number, if present. e.g. 00,001.
  • Tweak lookup code to support translated messages as JSON rather than code.
  • Update data to CLDR 31.0.1
  • Adds locales en_MY, fr_CH, it_CH, and ps.
  • Use locale digits for printing DateTime. This can also be disabled for a
    particular locale use useNativeDigitsByDefaultFor or for a particular
    DateFormat instance use useNativeDigits.
  • Provide a library for custom-initialized DateTime and number formatting. This
    allows easier custom locales and synchronous initialization.

@rodydavis
Copy link
Contributor Author

looks like it was mostly bug fixes but Remove deprecated calls, could be an issue.

@amirh
Copy link
Contributor

amirh commented Sep 24, 2019

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.

@amirh
Copy link
Contributor

amirh commented Sep 25, 2019

Seems like the major version bump for the intl package was a mistake, we should use >=0.15.1 <0.17.0 as the intl constraint here and make this a patch version bump.

@rodydavis
Copy link
Contributor Author

Ok sounds great 👍🏼

@rodydavis
Copy link
Contributor Author

OK i just updated it to use >=0.15.1 <0.17.0

author: Flutter Team <[email protected]>
homepage: https://github.com/flutter/plugins/tree/master/packages/local_auth
version: 0.6.0
version: 0.6.1
Copy link
Contributor

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed an update

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@amirh amirh merged commit 9fa4e02 into flutter:master Sep 26, 2019
@amirh amirh mentioned this pull request Sep 26, 2019
13 tasks
mormih pushed a commit to mormih/plugins that referenced this pull request Nov 17, 2019
sungmin-park pushed a commit to sungmin-park/flutter-plugins that referenced this pull request Dec 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants