Skip to content

feat: add yaml file extension support for parsing#13638

Open
SFL79 wants to merge 2 commits intoinfiniflow:mainfrom
SFL79:feat/add-yaml-extension
Open

feat: add yaml file extension support for parsing#13638
SFL79 wants to merge 2 commits intoinfiniflow:mainfrom
SFL79:feat/add-yaml-extension

Conversation

@SFL79
Copy link
Contributor

@SFL79 SFL79 commented Mar 16, 2026

What problem does this PR solve?

RAGFlow declines .yaml extension files and only accepts .yml files

Type of change

  • Bug Fix (non-breaking change which fixes an issue)

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Mar 16, 2026
@SFL79
Copy link
Contributor Author

SFL79 commented Mar 16, 2026

This PR ends up fixing 2 issues:

  1. ragflow declines yaml but accepts yml in upload stage.
  2. ragflow didnt have a matching parser for yml/yaml extensions so even a yml file ended up being unable to get parsed. I added yaml and yml files to the txtParser. Also, made sure the error ragflow throws is up to date with all file extensions checked in the code block

Im actually debating on this with myself as yaml might benefit more from a different kind of chunking, however it is better than the existing condition where there's none. Let me know what you think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant