-
Notifications
You must be signed in to change notification settings - Fork 8
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
update Readme to document Future, async and await #10
Conversation
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.
@SimonLab In my opinion and a very interesting piece of information that should certainly be added.
} | ||
``` | ||
|
||
To test this code you can copy/paste it and run it on dartpad: https://dartpad.dev/ |
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.
👍
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.
@SimonLab great additions. 🥇
I've made a few minor suggestions. ✍️
Feel free to accept/reject .then
we can merge. 😉
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 for another set of great additions @SimonLab, learning a lot from reviewing these PRs!
Thanks for fixing the `futur` typo! Co-Authored-By: Nelson <[email protected]>
Update Future states description (uncompleted, completed success, completed error) Co-Authored-By: Nelson <[email protected]>
Thanks for the reviews! @nelsonic I've merged your suggestions and it should be ready to merge now 👍 |
@SimonLab Thanks for accepting the suggestions. |
ref: #9
Add an
asynchronous
section, to explain how to use theFuture
class and theasync/await
andthen
keywords