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 for Issue #616: Handle Ingestion of Documents with Uppercase Extensions #621

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

Conversation

fshnkarimi
Copy link

Hello,

I've addressed the issue #616 regarding the problem faced when ingesting documents with uppercase file extensions. Here's a breakdown of my changes:

Modified the load_documents function: now, files with uppercase extensions are renamed to have lowercase extensions before processing.
This ensures that documents, regardless of their original extension case, are ingested consistently and without issues.
Checked that all changes are in compliance with the pre-commit hooks and confirmed they pass all checks.
By making file extensions uniformly lowercase, we should now be able to ingest documents without facing issues related to varied extension casing.

Please review my pull request, and I'd appreciate any feedback or suggestions for further refinement.

Thank you for the opportunity to contribute!

Warm regards,
Afshin Karimi

@LeafmanZ
Copy link
Collaborator

LeafmanZ commented Dec 6, 2023

works well

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.

None yet

2 participants