-
Notifications
You must be signed in to change notification settings - Fork 0
Update README with TODO and KBUtilLib note #2
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
base: develop
Are you sure you want to change the base?
Conversation
Added TODO section and updated KBUtilLib description.
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.
Pull request overview
This PR adds a TODO section to the README and marks the KBUtilLib utilities as a pending item. The changes provide visibility into planned improvements and clarify the status of certain dependencies.
- Added a TODO section noting the investigation of DuckDB or in-memory database options
- Updated the KBUtilLib utilities feature to indicate it's a TODO item
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| FastAPI application for table scanning operations with MinIO storage integration. | ||
|
|
||
| # TODO | ||
| * Look into duckdb or in memory db |
Copilot
AI
Dec 15, 2025
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 spelling of "duckdb" is incorrect. It should be capitalized as "DuckDB" which is the proper name of the database system.
| * Look into duckdb or in memory db | |
| * Look into DuckDB or in memory db |
| FastAPI application for table scanning operations with MinIO storage integration. | ||
|
|
||
| # TODO | ||
| * Look into duckdb or in memory db |
Copilot
AI
Dec 15, 2025
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 phrase "in memory db" should be hyphenated as "in-memory db" or better yet, "in-memory database" for consistency with professional technical writing standards.
| * Look into duckdb or in memory db | |
| * Look into duckdb or in-memory database |
| # TODO | ||
| * Look into duckdb or in memory db | ||
|
|
||
|
|
Copilot
AI
Dec 15, 2025
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 are excessive blank lines after the TODO section. Consider removing one of the two consecutive blank lines to maintain consistent spacing throughout the document.
| - MinIO client integration | ||
| - KBUtilLib utilities | ||
| - KBUtilLib utilities (TODO) | ||
|
|
Copilot
AI
Dec 15, 2025
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.
Trailing whitespace has been added on this line. This should be removed to maintain clean formatting and avoid unnecessary whitespace in the document.
Added TODO section and updated KBUtilLib description.