Skip to content

Feat/version - #117

Merged
balajz merged 7 commits into
mainfrom
feat/version
Aug 20, 2026
Merged

Feat/version#117
balajz merged 7 commits into
mainfrom
feat/version

Conversation

@balajz

@balajz balajz commented Aug 20, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI lite review requested due to automatic review settings August 20, 2026 14:12

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

Introduces a centralized build-time version metadata mechanism for pgxcli, embedding version/commit/build-time into the binary and updating CLI/release tooling to surface and populate that information consistently.

Changes:

  • Add internal/version package with linker-injected Version, Commit, and BuildTime plus a full version formatter.
  • Update the Cobra root command to use the shared version values and print a richer version output.
  • Wire version metadata into builds via Makefile, GoReleaser config, and the GitHub Actions release workflow.

Reviewed changes

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

Show a summary per file
File Description
VERSION Adds a single source version value for local builds.
Makefile Injects version/commit/build time via -ldflags into make build.
internal/version/version.go New package holding version metadata and formatting helper.
internal/cli/root.go Uses shared version metadata and custom Cobra version template.
.goreleaser.yaml Updates ldflags to inject version metadata into release binaries.
.github/workflows/release.yml Updates Windows release build to inject version metadata (including build time).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Makefile Outdated
balajz and others added 2 commits August 20, 2026 19:51
Signed-off-by: balaji j <balaji.jothi.in@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@balajz
balajz requested a balanced review from Copilot August 20, 2026 14:26

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@balajz
balajz merged commit 67c7d90 into main Aug 20, 2026
7 of 8 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