-
-
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
Improve documentation and examples #30
Conversation
2baedd5
to
e1de397
Compare
0610d18
to
ed34315
Compare
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.
Looks good to me 👍
ed34315
to
42941db
Compare
5a1aa65
to
6f7eea6
Compare
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.
@yadaiio Looks good so far, the documentation for the Transformer can also be found again inside the Transformer.php
file, makes sense to update this as well.
6f7eea6
to
5d408b9
Compare
Thank you @SimonFrings for keeping an eye out for this. 👍 |
34a5a60
to
e9fcb23
Compare
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.
Looks good to me 👍
e9fcb23
to
8b9aca0
Compare
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.
@yadaiio Thanks for the update, changes LGTM!
Builds on top of #28, #27 and #24.
References: clue/reactphp-tar#20, clue/reactphp-redis#117, clue/reactphp-shell#20 and clue/reactphp-sqlite#43.