Skip to content

chore: remove cgo since we removed pg_query_go - #119

Merged
tianzhou merged 1 commit into
mainfrom
simplify
Oct 23, 2025
Merged

tianzhou merged 1 commit into
mainfrom
simplify

Conversation

@tianzhou

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings October 23, 2025 10:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes CGO dependencies from the build process following the removal of the pg_query_go dependency. The changes simplify the build configuration by eliminating CGO-specific tooling and switching to native Go cross-compilation.

Key Changes:

  • Removed CGO build tools (gcc, g++, musl-dev) and runtime dependencies (libc6-compat) from Docker configuration
  • Replaced Docker-based cross-compilation with native Go cross-compilation in CI workflows
  • Updated documentation to reflect CGO-free build commands

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
Dockerfile Removed CGO build dependencies and runtime libraries, disabled CGO_ENABLED flag
CLAUDE.md Removed CGO_ENABLED=1 from build command documentation
.github/workflows/release.yml Replaced Docker/QEMU-based builds with native Go cross-compilation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@tianzhou
tianzhou merged commit 569d175 into main Oct 23, 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.

2 participants