Skip to content

Added the search cli command#43

Merged
PaoloLeonard merged 49 commits intomainfrom
dev
Jul 1, 2025
Merged

Added the search cli command#43
PaoloLeonard merged 49 commits intomainfrom
dev

Conversation

@ChristopheBeke
Copy link
Contributor

@ChristopheBeke ChristopheBeke commented Jul 1, 2025 •
edited
Loading

Pull Request

Description

Added the search cli command, including tests and new export writer.

Fixes # (issue)

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Checklist

  • My code follows the style guidelines of this project
  • 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
  • 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
  • Any dependent changes have been merged and published in downstream modules

ChristopheBeke and others added 30 commits June 15, 2025 15:35
@ChristopheBeke
Added the docbinder search command that allows to search through all …
9a5ee17 
…google drive documents
@ChristopheBeke
Changed all commands to dedicated folders and better structure. Also …
3722d39 
…added tests for the new search functionality.
@ChristopheBeke
Added traversal of child directories to get all the files
a9fc52c
@ChristopheBeke
Delete a file that wasn't supposed to be there
178ea9a
@ChristopheBeke
Update tests
7260899
@ChristopheBeke
Merge branch 'main' of https://github.com/SnappyLab/DocBinder-OSS int…
60d580a 
…o 2-add-the-capability-to-search-for-documents-metadata
@ChristopheBeke
Fixed merge conflict in 'src/docbinder_oss/services/google_drive/goog…
d107a8f 
…le_drive_client.py' of the docbinder-oss repository.
@ChristopheBeke
Add the path of the items as an attribute.
9e5a51b
@ChristopheBeke
Update src/docbinder_oss/services/google_drive/google_drive_client.py
c753807 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ChristopheBeke
Update src/docbinder_oss/services/google_drive/__init__.py
9dbf490 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ChristopheBeke
Update tests/services/test_search_export.py
98a6bc8 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ChristopheBeke
Removed unused parameter
7747680
@PaoloLeonard
Merge branch 'dev' of https://github.com/SnappyLab/DocBinder-OSS into…
461bc22 
… 2-add-the-capability-to-search-for-documents-metadata
@PaoloLeonard
improved help message and internal logic
59b00ff
@PaoloLeonard
typo in list
3da72b5
@PaoloLeonard
improved the internal logic
6e68c57
@PaoloLeonard
change to the provdier
b4e2096
@PaoloLeonard
improved the cli
eebc5b5
@PaoloLeonard
refactoring list all
ea4ebfc
@ChristopheBeke
Merge pull request #32 from SnappyLab/bugfix/2-add-the-capability-to-…
e187467 
…search-for-documents-metadata-paolo

Bugfix/2 add the capability to search for documents metadata paolo
@ChristopheBeke
Update with all new comments. Also made sure the search command now f…
a5d2e6b 
…ully adds all required fields for csv and json.
@PaoloLeonard
initial change to gdrive
63bd58c
@PaoloLeonard
Merge branch '2-add-the-capability-to-search-for-documents-metadata' of
b960941 
https://github.com/SnappyLab/DocBinder-OSS into improve-gdrive-integration
@PaoloLeonard
refactor google drive
b2d8d87
@PaoloLeonard
increased the page size
36fa20d
@PaoloLeonard
ruff linting
a475692
@PaoloLeonard
Merge branch 'improve-gdrive-integration' of https://github.com/Snapp…
2c06ddb 
…yLab/DocBinder-OSS into 36-add-better-writer-support
@PaoloLeonard
added nice writers and printing
2c5718f
@PaoloLeonard
corrected testsé
ec57051
@ChristopheBeke
Merge pull request #37 from SnappyLab/36-add-better-writer-support
c93b6e8 
36 add better writer support
ChristopheBeke and others added 17 commits June 27, 2025 18:15
@ChristopheBeke
Changed filter_files to private method and updated linting.
8a958e1
@ChristopheBeke
Added black for formatting and ruff for linting, including pre-commits
01f451e
@ChristopheBeke
Added pre-commit
97d749e
@ChristopheBeke
Update contributing file
8c1fd29
@ChristopheBeke
Merge pull request #35 from SnappyLab/improve-gdrive-integration
19e06a1 
Improve gdrive integration
@ChristopheBeke
Fixed the search cli --export-filename and improved the writer functi…
c31d4bb 
…on to now show which extensions are supported.
@ChristopheBeke
Updated the writer functions to work and improve readibility and alig…
27a1b5d 
…n the tests with it.
@ChristopheBeke
Make sure to get all files, not only the shared ones
c7747b2
@ChristopheBeke
Fix mkdocs
463ce84
@ChristopheBeke
Update incorrect readme reference in mkdocs
4827ce9
@ChristopheBeke
update workflow of docbinder oss to not trigger on doc updates and ch…
da02887 
…anged the reference name of docbinder documentation to be more correct
@PaoloLeonard
revert back to writer and improve tests
ca62bc6
@PaoloLeonard
remove logger
090ee29
@PaoloLeonard
fix linting
a803975
@ChristopheBeke
Merge pull request #42 from SnappyLab/revert-back-to-writer-and-impro…
d4d2388 
…ve-test

Revert back to writer and improve test
@ChristopheBeke
Merge pull request #25 from SnappyLab/2-add-the-capability-to-search-…
d4a8f3d 
…for-documents-metadata

2 add the capability to search for documents metadata
@ChristopheBeke
Merge branch 'main' of https://github.com/SnappyLab/DocBinder-OSS int…
2b40c0a 
…o dev
PaoloLeonard
PaoloLeonard requested changes Jul 1, 2025
View reviewed changes
@PaoloLeonard PaoloLeonard self-requested a review July 1, 2025 07:44
@PaoloLeonard PaoloLeonard merged commit 3e825ac into main Jul 1, 2025
3 checks passed
@ChristopheBeke ChristopheBeke deleted the dev branch July 1, 2025 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PaoloLeonard PaoloLeonard PaoloLeonard approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ChristopheBeke @PaoloLeonard