Skip to content

Conversation

@muhlba91
Copy link
Owner

@muhlba91 muhlba91 commented Oct 3, 2025

No description provided.

@coderabbitai
Copy link

coderabbitai bot commented Oct 3, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Introduces a build argument POSTGRESQL_MAJOR_VERSION=16 and updates the VectorChord download/install step in the Dockerfile to reference this argument in the package name and URL, parameterizing the PostgreSQL major version used during build.

Changes

Cohort / File(s) Change Summary
Parameterize PostgreSQL version in build
Dockerfile
Added ARG POSTGRESQL_MAJOR_VERSION=16. Updated RUN step to use ${POSTGRESQL_MAJOR_VERSION} in VectorChord package name and download URL, replacing hardcoded postgresql-16-... references.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/tag-naming

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 85d32aa and 612788d.

⛔ Files ignored due to path filters (2)
  • .github/workflows/release.yml is excluded by !**/*.yml
  • .github/workflows/verify.yml is excluded by !**/*.yml
📒 Files selected for processing (1)
  • Dockerfile (2 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gitstream-cm
Copy link

gitstream-cm bot commented Oct 3, 2025

Pull requests from the repository owner are approved automatically.

@muhlba91 muhlba91 merged commit 6e85db2 into main Oct 3, 2025
8 of 9 checks passed
@muhlba91 muhlba91 deleted the feat/tag-naming branch October 3, 2025 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants