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

Fix issue with future NFL boxscore errors #220

Merged
merged 1 commit into from
Oct 4, 2019

Conversation

roclark
Copy link
Owner

@roclark roclark commented Oct 4, 2019

For games that are yet to occur which are listed in an NFL team's schedule, an error will be thrown while attempting to parse a score for the game when none exists. To circumvent this issue, not only should the points be checked if they are None, they should also be checked if they are empty.

Fixes #219

Signed-Off-By: Robert Clark robdclark@outlook.com

For games that are yet to occur which are listed in an NFL team's
schedule, an error will be thrown while attempting to parse a score for
the game when none exists. To circumvent this issue, not only should the
points be checked if they are None, they should also be checked if they
are empty.

Signed-Off-By: Robert Clark <robdclark@outlook.com>
@roclark roclark added the bug Something isn't working label Oct 4, 2019
@roclark roclark added this to the Release 0.5.0 milestone Oct 4, 2019
@roclark roclark self-assigned this Oct 4, 2019
@codecov
Copy link

codecov bot commented Oct 4, 2019

Codecov Report

Merging #220 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
+ Coverage   99.97%   99.97%   +<.01%     
==========================================
  Files          42       42              
  Lines        9356     9357       +1     
==========================================
+ Hits         9354     9355       +1     
  Misses          2        2
Impacted Files Coverage Δ
sportsreference/nfl/boxscore.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 51df345...01d52e9. Read the comment docs.

@roclark roclark merged commit 3838679 into master Oct 4, 2019
@roclark roclark deleted the patch/fix-nfl-boxscore-errors branch October 4, 2019 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue pulling boxscore of future NFL games which haven't occurred
1 participant