Skip to content

Conversation

wxiao1002
Copy link
Contributor

pgvector_store not use PGVECTOR_CONNECTION_STRING from .env

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Snapshots:

Include snapshots for easier review.

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • 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
  • Any dependent changes have been merged and published in downstream modules

pgvector_store not use PGVECTOR_CONNECTION_STRING from .env
Aries-ckt
Aries-ckt previously approved these changes Sep 7, 2025
Copy link
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. thanks for your contribution.

@chenliang15405
Copy link
Collaborator

@wxiao1002 thanks for you contributtion. please fix code format, you can run command make fmt fix it

@chenliang15405 chenliang15405 changed the title pgvector_store can`t read connection_string from .env fix: pgvector_store read connection_string from env Oct 21, 2025
Copy link
Collaborator

@chenliang15405 chenliang15405 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot added the fix Bug fixes label Oct 21, 2025
Copy link
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Aries-ckt Aries-ckt merged commit 14f40dd into eosphoros-ai:main Oct 21, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants