run-llama / llama-hub

A library of data loaders for LLMs made by the community -- to be used with LlamaIndex and/or LangChain

https://llamahub.ai/

MIT License

3.43k stars 728 forks source link

use os.path.join to generate file paths #866

Closed Matt-Scheetz closed 6 months ago

Matt-Scheetz commented 6 months ago

Description

Utilize os.path.join to create OS agnostic file paths

Fixes # 865

Type of Change

Please delete options that are not relevant.

[ ] New Loader/Tool
[X] Bug fix / Smaller change
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Ran code locally to verify that files were successfully indexed using the AzStorageBlobReader

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[ ] I stared at the code and made sure it makes sense

Suggested Checklist:

[ ] I have added a library.json file if a new loader/tool was added
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods