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

Add warning that the shown README is not for the released version but for master #120

Merged
merged 1 commit into from
Oct 11, 2017
Merged

Add warning that the shown README is not for the released version but for master #120

merged 1 commit into from
Oct 11, 2017

Conversation

raarts
Copy link

@raarts raarts commented Oct 11, 2017

This bit me when I was trying to add tests, was scratching my head on why Tesla.Mock was not found.

@teamon
Copy link
Member

teamon commented Oct 11, 2017

The readme is for most recent version - 0.9.0 which does include Tesla.Mock. In fact, there is only a single commit on master after that release.

Please ensure you are using 0.9.0, if there is some issue with Tesla.Mock then this is definitely a bug.

@teamon teamon self-assigned this Oct 11, 2017
@coveralls
Copy link

coveralls commented Oct 11, 2017

Coverage Status

Coverage remained the same at 98.034% when pulling 1b48793 on raarts:warn-readme-is-newer-than-released into 59cfbdd on teamon:master.

@raarts
Copy link
Author

raarts commented Oct 11, 2017

Ah, I see that you released 0.9.0 5 days ago. My problem with Tesla.Mock stems from a week ago. Anyway, I think the warning is still valid, and the README missed a link to the docs at the top.

@teamon
Copy link
Member

teamon commented Oct 11, 2017

Ah, ok. Sure, it makes sense to add this warning. Thanks!

@teamon teamon merged commit ad0a7a3 into elixir-tesla:master Oct 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants