Skip to content

Made power rankings get current week by default and week parameter optional#32

Open
dtcarls wants to merge 1 commit into
rbarton65:masterfrom
dtcarls:master
Open

Made power rankings get current week by default and week parameter optional#32
dtcarls wants to merge 1 commit into
rbarton65:masterfrom
dtcarls:master

Conversation

@dtcarls

@dtcarls dtcarls commented Aug 23, 2017

Copy link
Copy Markdown

Attempting to resolve #31 Please double check functionality, not sure how to thoroughly test.

@dtcarls

dtcarls commented Aug 28, 2017

Copy link
Copy Markdown
Author

Tested as follows.

$ python3 setup.py install
$ python3
>>> from espnff import League
>>> league_id = 164483
>>> year = 2016
>>> league = League(league_id, year)
>>> league.get_current_week()
18
>>> year = 2017
>>> league = League(league_id, year)
>>> league.get_current_week()
1

@dtcarls

dtcarls commented Sep 12, 2017

Copy link
Copy Markdown
Author

Update to verify it works correctly going into week 2

$ python3
>>> from espnff import League
>>> league = League(164483, 2017)
>>> league.get_current_week();
2

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.

Request: Power rankings take week as optional argument.

1 participant