Skip to content

Propose version suffix strategy for non-git snap builds#547

Closed
jonathan-conder with Copilot wants to merge 1 commit into
fix/snapcraft-optional-gitfrom
copilot/sub-pr-546
Closed

Propose version suffix strategy for non-git snap builds#547
jonathan-conder with Copilot wants to merge 1 commit into
fix/snapcraft-optional-gitfrom
copilot/sub-pr-546

Conversation

Copilot AI commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

Addresses feedback on handling snap version suffixes when building outside git context (e.g., from release tarballs).

Current behavior:

  • Git repo: VERSION-<hash> for traceability
  • Git worktree: VERSION-dev for development builds
  • Non-git context: VERSION (clean semantic version)

Recommendation

No suffix for non-git builds. Release tarballs already contain the official version from VERSION file. Git-based suffixes serve development/testing traceability; release builds don't need additional markers.

Alternative

If explicit handling is preferred: -source over -nogit. Indicates "built from source tarball" rather than exposing implementation details.

Awaiting maintainer preference before implementing changes.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Address feedback on git shortref suffix in snap build process Propose version suffix strategy for non-git snap builds Nov 21, 2025
@jonathan-conder

Copy link
Copy Markdown
Contributor

No suffix for non-git builds. Release tarballs already contain the official version from VERSION file. Git-based suffixes serve development/testing traceability; release builds don't need additional markers.

Yes this was basically my initial reasoning

@jonathan-conder jonathan-conder deleted the copilot/sub-pr-546 branch November 21, 2025 05:06
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