search

zulip / zulip-flutter

Upcoming Zulip mobile apps for Android and iOS, using Flutter
Apache License 2.0
197 stars 192 forks source link

deps: Upgrade Flutter; and handle migrations #1057

Closed rajveermalviya closed 1 week ago

rajveermalviya commented 2 weeks ago

deps: Upgrade Flutter to 3.27.0-1.0.pre.443

And update Flutter's supporting libraries to match.

deps: Update CocoaPods to 1.16.2

Changelog: https://github.com/CocoaPods/CocoaPods/blob/master/CHANGELOG.md#1162-2024-10-31

Upstream changed the recommended CocoaPods version to 1.16.2 in: https://github.com/flutter/flutter/commit/dff191625f213a7cfdd1731d6f7b695e638ebfd2

Without this change flutter run generates the following warnings:

Warning: CocoaPods recommended version 1.16.2 or greater not installed.
Pods handling may fail on some projects involving plugins.
To update CocoaPods, see https://guides.cocoapods.org/using/getting-started.html#updating-cocoapods

intl: Migrate off of synthetic package; check-in generated files

flutter tool deprecated the package:flutter_gen synthetic package: https://flutter.dev/to/flutter-gen-deprecation

So, instead migrate to referencing source files directly.

Without this change, flutter run generates the following warning:

Synthetic package output (package:flutter_gen) is deprecated: https://flutter.dev/to/flutter-gen-deprecation. In a future release, synthetic-package will default to `false` and will later be removed entirely.
gnprice commented 1 week ago

Merging the first two commits now, to have the Flutter upgrade in when I make a new beta release shortly.

@rajveermalviya please go ahead and send the last commit: 6a495027d intl: Migrate off of synthetic package; check-in generated files

as a follow-up PR, after handling the comments above. Thanks again!

gnprice commented 1 week ago

(This means we'll have that warning appear on flutter run for the moment:

Synthetic package output (package:flutter_gen) is deprecated:
https://flutter.dev/to/flutter-gen-deprecation. In a future release, synthetic-package will default
to `false` and will later be removed entirely.

but that's tolerable, and we'll take care of it soon.)

rajveermalviya commented 1 week ago

send the last commit … as a follow-up PR

@gnprice Sent #1058.