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

Add include_today option to HistoricalReader #276

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lsapan
Copy link

@lsapan lsapan commented Apr 5, 2022

As the title says, just a quick change to allow passing include_today=True to get_historical_data and HistoricalReader. Thanks for a great library!

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

@addisonlynch
Copy link
Owner

Thanks. I haven't had much time to look at this library in a bit but I will take a look at this PR early next week.

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