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

Update README to avoid error on route in Sinatra #2

Merged
merged 1 commit into from
Sep 28, 2017

Conversation

gmmcal
Copy link
Contributor

@gmmcal gmmcal commented Sep 27, 2017

Since Goatmail is only available in development, if you mount the route
/inbox in other environment it will generate an error

config.ru:in `block (2 levels) in <main>':
  uninitialized constant Goatmail (NameError)

This fixes the instructions on how to mount the route properly.

Since Goatmail is only available in development, if you mount the route
`/inbox` in other environment it will generate an error

```
config.ru:in `block (2 levels) in <main>':
  uninitialized constant Goatmail (NameError)
```

This fixes the instructions on how to mount the route properly.
@coveralls
Copy link

coveralls commented Sep 27, 2017

Coverage Status

Coverage decreased (-1.8%) to 48.214% when pulling 578ed3d on gmmcal:fix-sinatra-readme into f3d1b9e on tyabe:master.

@gmmcal
Copy link
Contributor Author

gmmcal commented Sep 27, 2017

wow not sure how a change on README would decrease coverage 😂

@tyabe tyabe merged commit c5e0ddc into tyabe:master Sep 28, 2017
@tyabe
Copy link
Owner

tyabe commented Sep 28, 2017

Thanks!
I’m not sure, how to test for README ¯_(ツ)_/¯

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

3 participants