Skip to content

Conversation

@sanchitram1
Copy link
Contributor

There are some redundant merge requests in here, but:

  1. fix some init-script.sql stuff related to deployment
  2. fix a requirement to use psycopg2-binary instead of psycopg2
  3. use uv to build the docker images for the indexers as well, similar to how we do it for alembic

@jhheider jhheider requested a review from Copilot August 11, 2025 14:06
Copy link
Contributor

Copilot AI left a 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 performs maintenance tasks to improve the project's dependency management and Docker build process. The changes focus on switching from psycopg2 to psycopg2-binary for PostgreSQL connectivity and standardizing Docker builds to use uv for faster package installation.

  • Replace psycopg2 with psycopg2-binary for better deployment compatibility
  • Migrate package manager Docker images from pip to uv for consistent build tooling
  • Fix documentation and database initialization script issues

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pyproject.toml Updates psycopg2 dependency to use binary version with pinned version
core/requirements.txt Reflects the psycopg2-binary dependency change from pyproject.toml
package_managers/*/Dockerfile Migrates from python:3.11 base image to uv-enabled image with uv pip install
alembic/init-script.sql Enables pg_trgm extension instead of commented pg_bigm
alembic/Dockerfile Adds TODO comment about psql dependency requirement
README.md Corrects typo in dependency group name from "indexer" to "indexers"

"idna>=3.10",
"permalint>=0.1.15",
"psycopg2>=2.9.10",
"psycopg2-binary==2.9.10",
Copy link

Copilot AI Aug 11, 2025

Choose a reason for hiding this comment

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

Changing from a range constraint (>=2.9.10) to an exact pin (==2.9.10) removes flexibility for patch updates that might contain security fixes. Consider using a more flexible constraint like ~=2.9.10 to allow patch-level updates.

Suggested change
"psycopg2-binary==2.9.10",
"psycopg2-binary~=2.9.10",

Copilot uses AI. Check for mistakes.
@jhheider jhheider merged commit b05aae6 into teaxyz:main Aug 11, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants