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 the nbr of strategies in first page of docs #1321

Merged
merged 2 commits into from
Apr 8, 2020

Conversation

drvinceknight
Copy link
Member

Running len(axelrod.strategies) gives 235 :)

Genuinely not sure if I've got my grammar right in this sentence so please do suggest something else. I just wanted to bump the number :)

Running `len(axelrod.strategies)` gives 235 :)
docs/index.rst Outdated
@@ -8,7 +8,7 @@ Welcome to the documentation for the Axelrod Python library

Here is quick overview of the current capabilities of the library:

* Over 100 strategies from the literature and some exciting original
* Over 230 strategies including from the literature and some exciting original
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

including many?

I had a thought that we could add a classifier, something like "from a publication" (not necessary for this PR, of course).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

including many?

Thank you! Much better.

I had a thought that we could add a classifier, something like "from a publication" (not necessary for this PR, of course).

Not a bad idea and relatively straightforward with @gaffney2010 recent classifier work 👍

@meatballs meatballs merged commit 1ec7569 into master Apr 8, 2020
@meatballs meatballs deleted the update-number-of-strategies-in-docs branch April 8, 2020 16:02
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.

None yet

3 participants