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 KeyError exception when there is no earnings or financials data on a ticker. #60

Merged
merged 3 commits into from
May 22, 2018

Conversation

reixd
Copy link

@reixd reixd commented May 21, 2018

Deliver an empty list to denote the missing iterable data.

  • tests added / passed
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • added entry to docs/source/whatsnew/vLATEST.txt

@codecov
Copy link

codecov bot commented May 21, 2018

Codecov Report

Merging #60 into master will increase coverage by 0.07%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
+ Coverage   96.24%   96.31%   +0.07%     
==========================================
  Files          13       13              
  Lines        1596     1601       +5     
==========================================
+ Hits         1536     1542       +6     
+ Misses         60       59       -1
Impacted Files Coverage Δ
iexfinance/stock.py 93.33% <100%> (ø) ⬆️
tests/test_stock.py 100% <100%> (ø) ⬆️
iexfinance/__init__.py 98.48% <100%> (ø) ⬆️
tests/test_market.py 95.58% <0%> (+1.47%) ⬆️

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 c723397...53848f9. Read the comment docs.

@addisonlynch
Copy link
Owner

Thanks. Can you rebase please?

@reixd reixd changed the base branch from master to dev May 22, 2018 17:37
@reixd
Copy link
Author

reixd commented May 22, 2018

@addisonlynch Done

@addisonlynch addisonlynch merged commit dba90a4 into addisonlynch:dev May 22, 2018
@addisonlynch
Copy link
Owner

addisonlynch commented May 22, 2018

@reixd actually if you get the chance would you mind creating a new pull to master instead of dev? I hadn't looked at the dev branch and it needs to be deleted, has a bunch of messed up commits. Sorry about that.

@addisonlynch
Copy link
Owner

@reixd I took care of it actually.

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