-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Samples don't run with latest Flutter #208
Comments
@brianegan, are they any plans for migration of the code base to the latest Flutter, including null-safety ? |
@StanevPrime Thanks for writing in! Was busy for holidays & catching up on work. Right now, I've been going through the following plan:
However, long-term I wonder if this project should actually change a bit. It's been very hard to keep each app up-to-date, not only with Null-safety but also with changes to the libraries themselves. Many of the samples are still perfectly valid, but some of them, such as the Instead, I wonder if this project should merely export the core packages listed above, and any package could implement a "Flutter Architecture Sample Todo App" by writing a Flutter app that passes the integration tests suite. Then, any author could merely submit a pull request to this project to update the README pointing to the Todo app in their own repository. What do you think? |
This is great idea! Speaking of scalability and maintainability challenges this approach will definitely help in long-term. A assume there will be two packages
and Looking into the bloc library there is already a self-hosted copy of the todos sample, which in future can install and use those packages as follows
Also you can go further by uploading those packages to pub.dev so that it will be easier for each sample to adapt any breaking changes. |
Hello This does not support Flutter's latest version.
etc
|
Hey, any updated on this? |
Many (if not all) samples do not compile/run with the latest Flutter (Flutter 2.5.0).
One set of issues is related to the change of font names. See open PR.
The text was updated successfully, but these errors were encountered: