Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Update to allow intl in pubspec.yaml to be imported <= version 0.17.0… #603

Conversation

ebelliveau
Copy link

This is to enable compatibility within any apps requiring both firebase_auth^1.0.0 and charts_flutter when packaging the two components in the same app.

For example, when building a project which depends on the latest charts_flutter^0.9.0 and firebase_auth^1.0.0:

PS C:\Users\SomeUser\Development\someApp> flutter pub get
Because no versions of charts_flutter match >0.9.0 <0.10.0 and charts_flutter 0.9.0 depends on intl >=0.15.2 <0.17.0, charts_flutter ^0.9.0 requires intl >=0.15.2 <0.17.0.
And because firebase_auth >=1.0.0 depends on firebase_auth_web ^1.0.0 which depends on intl ^0.17.0, charts_flutter ^0.9.0 is incompatible with firebase_auth >=1.0.0.
So, because someApp depends on both firebase_auth ^1.0.0 and charts_flutter ^0.9.0, version solving failed.
Running "flutter pub get" in someApp...
pub get failed (1; So, because someApp depends on both firebase_auth ^1.0.0 and charts_flutter ^0.9.0, version solving failed.)

It's just a single comparator in the version designation (<= vs <), and the tests continute to pass, so I hope this is just a small omission/typo in version pinning between two (Google) libraries.

Closes #602

…, rather than < 0.17.0 to fix firebase_auth dependency on intl library.
@google-cla google-cla bot added the cla: yes label Mar 8, 2021
@ebelliveau
Copy link
Author

Ah no - I just saw the other PR to address this. Closing as there's already discussion for a few days in the other thread.

@ebelliveau ebelliveau closed this Mar 8, 2021
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.

Flutter 2.0 intl version compatible
1 participant