Conversation
jprakash-db
commented
Mar 20, 2026
- Updated some lines
- Setting up hatch
|
Thanks for your contribution! To satisfy the DCO policy in our contributing guide every commit message must include a sign-off message. One or more of your commits is missing this message. You can reword previous commit messages with an interactive rebase ( |
There was a problem hiding this comment.
Pull request overview
Updates the project README with additional sections, including a note about setting up the Python connector.
Changes:
- Added a new “Testing new lines” section to the README.
- Added a “How to setup the Python Connector” section mentioning Hatch-based setup.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Testing new lines | ||
| Added a new line describing the efforts |
There was a problem hiding this comment.
The new section title "Testing new lines" and the sentence below it read like placeholder text and don’t communicate actionable testing guidance. Consider removing this section or replacing it with actual test instructions (e.g., how to run the existing pytest suite).
| ## Testing new lines | ||
| Added a new line describing the efforts | ||
|
|
||
| ## How to setup the Python Connector |
There was a problem hiding this comment.
Header uses "setup" as a verb; standard usage is "set up" (two words) for the action. Please adjust the heading to "How to set up the Python Connector" for correctness and consistency.
| ## How to setup the Python Connector | |
| ## How to set up the Python Connector |
| Added a new line describing the efforts | ||
|
|
||
| ## How to setup the Python Connector | ||
| Python Connector needs to setup using hatch |
There was a problem hiding this comment.
This line says the connector needs to be set up using Hatch, but the repository’s pyproject.toml is configured for Poetry (no Hatch configuration present). Either update the docs to reflect Poetry-based setup, or add the missing Hatch configuration/instructions if Hatch is intended going forward.
| Added a new line describing the efforts | ||
|
|
||
| ## How to setup the Python Connector | ||
| Python Connector needs to setup using hatch |
There was a problem hiding this comment.
Grammar/formatting: "needs to setup using hatch" should be "needs to be set up using Hatch" (and remove the double space).
| Python Connector needs to setup using hatch | |
| The Python connector needs to be set up using Hatch. |