-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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: change DirectoryComponent to filter file paths by types #2391
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The component is wrong I think. The types
field should be a list field. That's why the self.types requires a call to split
but shouldn't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ogabrielluiz Please verify the corrections applied.
…nd adding a comment for type hint ignore in ChatGroq instantiation.
* Refactored mutation and query types and callables * Refactored version and other queries to new way of calling function * update type declaration to support options * update getVersionQuery * [autofix.ci] apply automated fixes * update type declaration to remove on Fetch options * remove onFetch from version * update transactions query * [autofix.ci] apply automated fixes --------- Co-authored-by: anovazzi1 <[email protected]> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: cristhianzl <[email protected]>
* opentelemetry and prometheus * set env override in create_app * add prometheus_client to pyproject * update top level poetry.lock
* update getHealth function to use options param * [autofix.ci] apply automated fixes --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
* fix version * [autofix.ci] apply automated fixes --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
chore: Add conditional step to delete-reports job in TypeScript test workflow
* fix: pasting files not working * [autofix.ci] apply automated fixes * refactor: change new import --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: cristhianzl <[email protected]> Co-authored-by: anovazzi1 <[email protected]>
…w-ai#2391) * Refactor DirectoryComponent to filter file paths by types * fix: sets types is_list to True * chore: Remove Optional from load_directory method signature * chore: Update return type annotation for load_directory method * style(GroqModel.py): improve code readability by fixing indentation and adding a comment for type hint ignore in ChatGroq instantiation. --------- Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]>
…w-ai#2391) * Refactor DirectoryComponent to filter file paths by types * fix: sets types is_list to True * chore: Remove Optional from load_directory method signature * chore: Update return type annotation for load_directory method * style(GroqModel.py): improve code readability by fixing indentation and adding a comment for type hint ignore in ChatGroq instantiation. --------- Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]> (cherry picked from commit bc6b846)
The "Type" option of the component has been adjusted to ensure the file type is selected correctly.