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

#47 Fixed bug where FakeRequest body was being written to the Response #48

Merged
merged 2 commits into from Jul 23, 2014

Conversation

Projects
None yet
4 participants
@carlosmiranda
Contributor

carlosmiranda commented Jul 23, 2014

No description provided.

@dmarkov

This comment has been minimized.

Show comment
Hide comment
@dmarkov

dmarkov Jul 23, 2014

Thanks for the pull request, let me find a reviewer..

dmarkov commented Jul 23, 2014

Thanks for the pull request, let me find a reviewer..

@dmarkov

This comment has been minimized.

Show comment
Hide comment
@dmarkov

dmarkov commented Jul 23, 2014

@yegor256 please review

@yegor256

This comment has been minimized.

Show comment
Hide comment
@yegor256
Member

yegor256 commented Jul 23, 2014

@rultor merge

@rultor

This comment has been minimized.

Show comment
Hide comment
@rultor

rultor Jul 23, 2014

Contributor

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

Contributor

rultor commented Jul 23, 2014

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 52e7b9b into jcabi:master Jul 23, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@rultor

This comment has been minimized.

Show comment
Hide comment
@rultor

rultor Jul 23, 2014

Contributor

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 3min)

Contributor

rultor commented Jul 23, 2014

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 3min)

@dmarkov

This comment has been minimized.

Show comment
Hide comment
@dmarkov

dmarkov Jul 25, 2014

@rultor please deploy

dmarkov commented Jul 25, 2014

@rultor please deploy

@rultor

This comment has been minimized.

Show comment
Hide comment
@rultor

rultor Jul 25, 2014

Contributor

@rultor please deploy

@dmarkov OK, I'll try to deploy now. You can check the progress here

Contributor

rultor commented Jul 25, 2014

@rultor please deploy

@dmarkov OK, I'll try to deploy now. You can check the progress here

@rultor

This comment has been minimized.

Show comment
Hide comment
@rultor

rultor Jul 25, 2014

Contributor

@rultor please deploy

@dmarkov Done! FYI, the full log is here (took me 7min)

Contributor

rultor commented Jul 25, 2014

@rultor please deploy

@dmarkov Done! FYI, the full log is here (took me 7min)

@dmarkov

This comment has been minimized.

Show comment
Hide comment
@dmarkov

dmarkov Jul 25, 2014

@yegor256 I just added 16 mins to your account, many thanks for your contribution..

dmarkov commented Jul 25, 2014

@yegor256 I just added 16 mins to your account, many thanks for your contribution..

@carlosmiranda carlosmiranda deleted the carlosmiranda:47 branch Jul 28, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment