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

Missing dependency to run tests locally #3320

Closed
stsewd opened this issue Nov 26, 2017 · 1 comment
Closed

Missing dependency to run tests locally #3320

stsewd opened this issue Nov 26, 2017 · 1 comment
Labels
Needed: documentation Documentation is required

Comments

@stsewd
Copy link
Member

stsewd commented Nov 26, 2017

Details

Hi, I was trying to run the tests locally following the guide, but I was getting an error, later I figure out that I needed mercurial installed, that requirement is not indicated on the guide.

Expected Result

Pass all the tests following the guide.

Actual Result

I was unable to pass the tests following the guide.

I would be happy to send a pull request later 😄

@RichardLitt
Copy link
Member

Sounds like this could be fixed with a small PR to note that mercurial should be installed! Looking forward to seeing that, @stsewd. Thanks for logging this.

@RichardLitt RichardLitt added the Needed: documentation Documentation is required label Nov 27, 2017
stsewd added a commit to stsewd/readthedocs.org that referenced this issue Dec 5, 2017
Some tests need mercurial to pass.

Closes readthedocs#3320
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: documentation Documentation is required
Projects
None yet
Development

No branches or pull requests

2 participants