Developers - how to contribute

[ValueRaider]

Branches

To support rapid development without breaking stable versions, this project uses a two-layer branch model:

(inspiration)

• dev - new features & some bug-fixes merged here, tested together, conflicts fixed, etc. Once stable, can merge to ...
• main: stable branch. Where PIP releases are created

Most of the time you want to target the dev branch, but default is main so remember to switch to dev.

Exception: can straight merge into main if:

• yfinance is massively broken
• or part of yfinance is broken and the fix is simple and clearly doesn't affect anything else

I'm a GitHub newbie, how do I contribute?

1. Fork this project. If already forked, remember to Sync fork
2. Implement your change in your fork, ideally in a specific branch
3. Create a Pull Request, from your fork to this project. If addressing an Issue, link to it

How to download & run a GitHub version of yfinance

Unit tests

Tests have been written using the built-in Python module unittest. Examples:

• Run all price tests: python -m unittest tests.test_prices

• Run sub-set of price tests: python -m unittest tests.test_prices.TestPriceRepair

• Run a specific test: python -m unittest tests.test_prices.TestPriceRepair.test_ticker_missing

• Run all tests: python -m unittest discover -s tests

Git stuff

commits

To keep the Git commit history and network graph compact please follow these two rules:

• For long commit messages use this: git commit -m "short sentence summary" -m "full commit message"
• squash tiny/negligible commits back with meaningful commits, or to combine successive related commits. Guide but basically it's:

git rebase -i HEAD~2
git push --force-with-lease origin <branch-name>

rebase

You might be asked to move your branch from main to dev. Make sure you have pulled all relevant branches then run:

git checkout <your branch>
git rebase --onto dev main <branch-name>
git push --force-with-lease origin <branch-name>

[jcenmiami]

Just an FYI, download Pandas 2.1.0 and I started getting iloc warnings while using YFinance (utils.py). It's just a matter of adding iloc[x] to the code. I added it to the library and the warnings disappeared. I downgraded to 1.5.3 and all is good while the next version of YFinance is ready. Thanks for everything, love this library.

[ValueRaider]

Is fixed in dev branch #1672. Simply haven't got round to releasing.

[s-kust]

How to run tests to make sure I don't break anything?

[ValueRaider]

Only just seen this. I've added a new section.

[busterbeam]

and it needs updating, as the tests still don't work properly. fresh clone and even after installing the extra packages not stated in requirements.txt does it do something. but yea, not a great start for any dev wanting to help.

[frommintoton]

I reinstall Anaconda oct 2024 version and install yfinance afterwards in one computer,

for some reason, df = yf.download(symbol) show a different outlook with
ticker label appears as an individual row
and the arrangement of Open, High, Low, Close becomes adj close, close, high, low, open

more important, df['Close'] > number does not work anymore as df['Close'] becomes a list type.

but we can still use:
df['OpentoClose'] = df['Open']/df['Close].shift(1)

it still works.