Skip to content

Add async gens for multiple clans/players#8

Merged
mathsman5133 merged 3 commits into
mathsman5133:masterfrom
JustinBacher:patch-1
Apr 22, 2019
Merged

Add async gens for multiple clans/players#8
mathsman5133 merged 3 commits into
mathsman5133:masterfrom
JustinBacher:patch-1

Conversation

@JustinBacher

Copy link
Copy Markdown
Contributor

I added these methods for:

  • get_clan -> get_clans
  • get_current_war -> get_current_wars
  • get_player -> get_players

I think that should be it. I also added examples in the docstrings as well. I figured collections.Iterable as a good enough description for people. And the tags = iter(tags) enforces that an iterable was passed before we even start iterating just as a check.

I added these methods for:

 - `get_clan` -> `get_clans`
 - `get_current_war` -> `get_current_wars`
 - `get_player` -> `get_players`

I think that should be it. I also added examples in the docstrings as well. I figured `collections.Iterable` as a good enough description for people. And the `tags = iter(tags)` enforces that an iterable was passed before we even start iterating just as a check.
@JustinBacher

JustinBacher commented Apr 22, 2019

Copy link
Copy Markdown
Contributor Author

Also added in the helper method here. Refer to #7

Comment thread coc/client.py Outdated
@mathsman5133 mathsman5133 merged commit 7cd1424 into mathsman5133:master Apr 22, 2019
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.

2 participants