Update datasources.py#294
Conversation
Updated to yfinance > 0.2.38; tested with 0.2.65
|
Hi, |
|
You're absolutely right. I messed that up and didn't fork it correctly. I will cancel this PR and submit a clean one. |
|
@chrislee35 You fork it correctly, there is, that you just continue working on branch which was connected to this PR. What should be done, is to open new branch let's say "fix-datasource-yfinance-api":
I don't know how you advanced in git, so please don't be angry on me if I told obviousness. 😃 |
|
I tried and now my repo is all broken. My git skills were unable to fix it. |
|
It's not look that bad, I see that you didn't lost your patch. Discord? |
|
How does this look? #295 |
Updated to yfinance > 0.2.38; tested with 0.2.65
Issue
This fixes issue #291.
yfinance changed its API. I updated datasources.py to the new API.
Description of changes
Includes