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

Remove attempted_passes from NCAAF #395

Merged
merged 1 commit into from
Apr 12, 2020
Merged

Conversation

roclark
Copy link
Owner

@roclark roclark commented Apr 12, 2020

The NCAAF module has deprecated the attempted_passes property in favor of pass_attempts to be more consistent with other modules as well as the naming convention already present in NCAAF.

Fixes #171

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

The NCAAF module has deprecated the attempted_passes property in favor
of pass_attempts to be more consistent with other modules as well as the
naming convention already present in NCAAF.

Signed-Off-By: Robert Clark <robdclark@outlook.com>
@roclark roclark added the deprecation warning Support for a language or feature will end label Apr 12, 2020
@roclark roclark added this to the Release 0.6.0 milestone Apr 12, 2020
@roclark roclark self-assigned this Apr 12, 2020
@codecov
Copy link

codecov bot commented Apr 12, 2020

Codecov Report

Merging #395 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #395      +/-   ##
==========================================
- Coverage   99.97%   99.97%   -0.01%     
==========================================
  Files          48       48              
  Lines       10582    10574       -8     
==========================================
- Hits        10579    10571       -8     
  Misses          3        3              
Impacted Files Coverage Δ
sportsreference/ncaaf/player.py 100.00% <ø> (ø)
sportsreference/ncaaf/roster.py 100.00% <ø> (ø)

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 9b93ce8...6149485. Read the comment docs.

@roclark roclark merged commit f5dd33d into master Apr 12, 2020
@roclark roclark deleted the remove-attempted-passes branch April 12, 2020 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecation warning Support for a language or feature will end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove attempted_passes from NCAAF
1 participant