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

Fix formatting in ConnectionFailed error message #805

Merged
merged 1 commit into from
Jun 28, 2019
Merged

Fix formatting in ConnectionFailed error message #805

merged 1 commit into from
Jun 28, 2019

Conversation

jcsrb
Copy link
Contributor

@jcsrb jcsrb commented Jun 28, 2019

Add space at the end of the second line so the complete message separates the word not and working

message before

Stripe::APIConnectionError: Unexpected error communicating when trying to connect to Stripe. You may be seeing this message because your DNS is notworking.  To check, try running `host stripe.com` from the command line.

message after

Stripe::APIConnectionError: Unexpected error communicating when trying to connect to Stripe. You may be seeing this message because your DNS is not working.  To check, try running `host stripe.com` from the command line.

Add space at the end of the second line so the complete message separates the word `not` and `working`

message before
```
Stripe::APIConnectionError: Unexpected error communicating when trying to connect to Stripe. You may be seeing this message because your DNS is notworking.  To check, try running `host stripe.com` from the command line.
```
message after
```
Stripe::APIConnectionError: Unexpected error communicating when trying to connect to Stripe. You may be seeing this message because your DNS is not working.  To check, try running `host stripe.com` from the command line.
```
@jcsrb
Copy link
Contributor Author

jcsrb commented Jun 28, 2019

jruby build fails, but it looks like an issue with RVM version, not related to my commit :|

Downloading https://github.com/rvm/rvm/archive/.tar.gz
curl: (22) The requested URL returned error: 404 Not Found

@brandur-stripe
Copy link
Contributor

Thank you @jcsrb!

And yeah, it looks like either RVM or JRuby is having issues right now. We're good to merge though.

@brandur-stripe brandur-stripe merged commit f4b874d into stripe:master Jun 28, 2019
@brandur-stripe
Copy link
Contributor

Released as 4.20.1.

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.

2 participants