dwyl / learn-flutter

🦋 Learn how to use Flutter to Build Cross-platform Native Mobile Apps
https://flutter.dev
GNU General Public License v2.0
73 stars 8 forks source link

Chore: Tidy up and Extend Learn Flutter Docs 📝 #68

Closed nelsonic closed 1 year ago

nelsonic commented 1 year ago

The contents of this repo is "OK" as a starting point but it feels like it could use some "polish" / updating.

Todo

Tidy Up

image
nelsonic commented 1 year ago

@LuchoTurtle if you can review this doc and update it, it would be a huge help! 🙏 I've already deleted my old ("stale") branch. ✂️ Please liberally update the grammar/style of writing to match our other projects. 📝 Anything else you think of along the way, comment! 💬

nelsonic commented 1 year ago

@LuchoTurtle's PR: https://github.com/dwyl/learn-flutter/pull/69 🎉

nelsonic commented 1 year ago

@LuchoTurtle to create a micro-PR to add GitHub Actions to this repo so that we can check everything is working. 🙏 As per: https://github.com/dwyl/flutter-counter-example/pull/7#issuecomment-1316625719

LuchoTurtle commented 1 year ago

@nelsonic you should find it here -> https://github.com/dwyl/learn-flutter/pull/71

nelsonic commented 1 year ago

Thanks. Merged. Please confirm Working. 🤞

LuchoTurtle commented 1 year ago

Similarly to what happened in https://github.com/dwyl/flutter-counter-example/issues/6#issuecomment-1316782820, the CI should work when working with the latest Flutter version, as it's the case with #69 . After this PR is merged, it should work 😄

nelsonic commented 1 year ago

Gotcha. Thanks. 👌