-
-
Notifications
You must be signed in to change notification settings - Fork 8
Improve documentation and examples #30
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
Conversation
2baedd5 to
e1de397
Compare
0610d18 to
ed34315
Compare
SimonFrings
left a comment
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
SimonFrings
left a comment
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
SimonFrings
left a comment
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
clue
left a comment
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.