Skip to content
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

Typo #1

Merged
merged 1 commit into from Jul 13, 2013
Merged

Typo #1

merged 1 commit into from Jul 13, 2013

Conversation

mykiwi
Copy link
Contributor

@mykiwi mykiwi commented Jul 13, 2013

No description provided.

@@ -82,7 +82,7 @@ private function send(Request $request)

$statusCode = $e->getResponse()->getStatusCode();
$reasonPhrase = $e->getResponse()->getReasonPhrase();
$message = sprintf('The request in not valid (status code: "%d", reason phrase: "%s").', $statusCode, $reasonPhrase);
$message = sprintf('The request is not valid (status code: "%d", reason phrase: "%s").', $statusCode, $reasonPhrase);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess you broke tests (I test exception message).
You can not see it as you did the patch on github ;)

I will add travis. Could you rebase after my commit for travis ?

Thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

lyrixx added a commit that referenced this pull request Jul 13, 2013
This PR was merged into the master branch.

Discussion
----------

Typo

Commits
-------

a7223f6 Typo
@lyrixx lyrixx merged commit a7223f6 into lyrixx:master Jul 13, 2013
@lyrixx
Copy link
Owner

lyrixx commented Jul 13, 2013

Oups, I merged you PR :(
I will fix the issue

@lyrixx
Copy link
Owner

lyrixx commented Jul 13, 2013

fixed by 46639cd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants