-
Notifications
You must be signed in to change notification settings - Fork 263
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
A script to automatically detect if all strategy modules are listed in docs/reference/all_strategies.rst
#957
Comments
@MariosZoulias perhaps you would like to take this one? |
Unless you want this @MariosZoulias I might pick it up tomorrow :) |
Hi . Sorry for answering so late . |
Ok sounds good, no problem about the late reply: good to know you want this issue :) 👍 |
@MariosZoulias I had a couple of minutes spare so I've picked this one up (#980) : hope that's ok :) |
Currently, whenever a new strategy module (file) is added to
axelrod/strategies
it needs to be also added todocs/reference/all_strategies.rst
to ensure that an index of all strategies is built as can be seen here: http://axelrod.readthedocs.io/en/latest/reference/all_strategies.htmlThis needs to be checked during the review process but it would be helpful to have a script that could be run (by travis) to indicate that all the modules have been listed. So for example it would look like follows if all were listed (no output):
Of if the
axelrod/strategies/tit_for_tat.py
file was not listed it would returnsys.exit(1)
(so that it's recognised as a failure):This could be done with an inspection of the
axelrod/strategies
directory as well as using some regex on thedocs/reference/all_strategies.rst
file.Note that we have explored automatically generating the
all_strategies.rst
file and came to a dead end (we can't get read the docs to do that for us for example).The text was updated successfully, but these errors were encountered: