-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat: upgrade project to work with the latest version of flutter #107
Conversation
Looks pretty good already! Can you fix the lint errors? |
I am seing some fairly strange behaavior where when I run |
I have figured it out. @kolaente I think it may be a better idea to put a step into the pipeline to do a pub get before running the format so the packages are formatted properly. What do you think? |
Format code repo after changes
a9844a2
to
929b69d
Compare
I took the liberty of tweaking the pipeline slightly to get packages before performing a format. Now it works without a problem. |
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.
Thanks!
thanks both! |
There are a few changes here all of which go towards making the project work with the latest version of flutter and compile properly. The summery of them are below
flutter pub upgrade --major versions
to upgrade dependencies inpubspec.yaml
notifications.yaml
and the file was updated accordinglyFlutter version:
Dart version:
Resolves #106