Skip to content

release: prepare v0.2.0 — CHANGELOG, chart bump, README link (refs #62) - #63

Merged
Coldwings merged 3 commits into
mainfrom
release/v0.2.0
Aug 26, 2026
Merged

release: prepare v0.2.0 — CHANGELOG, chart bump, README link (refs #62)#63
Coldwings merged 3 commits into
mainfrom
release/v0.2.0

Conversation

@Coldwings

Copy link
Copy Markdown
Collaborator

What

Prepares the v0.2.0 release (decision recorded in #62):

  1. CHANGELOG.md (new, tracked): full v0.2.0 entry —
  2. deploy/helm/dart/Chart.yaml: chart version 0.1.0 → 0.2.0, appVersion devv0.2.0 (first tag the chart can name).
  3. README.md: links CHANGELOG.md from the Documentation section.

The version string itself needs no code change: binaries stamp -ldflags -X main.version=... at build time.

After this merges, I will tag v0.2.0 (annotated) on the merge commit and create the GitHub Release with notes derived from the changelog, then close #62.

Refs #62

Verification

Docs/manifest-only change; still ran the full trio: gofmt clean; go vet ./... clean; go test ./... -race -count=1 13/13 PASS; -cover PASS.

Adds a tracked CHANGELOG.md (none existed) with the full v0.2.0 entry:
the four direct-commit additions (k8s EndpointSlice module + internal/node
extraction, Helm/Fluid samples, non-range passthrough, tracker idle
eviction), the 30 merged fix PRs grouped by subsystem, and the upgrade
notes with the wire-compat statement (Hash64/HRW score/ChunkKey/epoch
framing byte-identical to v0.1.0; cache-identity splits limited to
malformed URLs and duplicated static-peer IDs).

Chart: version 0.1.0 -> 0.2.0, appVersion dev -> v0.2.0 (first tag the
chart can name). README links the changelog from the Documentation
section.

Refs #62
@Coldwings

Copy link
Copy Markdown
Collaborator Author

@copilot please review. Focus on: factual accuracy of the changelog inventory against the merged PR list (#23-#44, #47, #48, #56-#61), the wire-compat upgrade-note claims, and the Chart.yaml version/appVersion conventions. Do NOT push commits; post review comments only.

Copilot AI commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

@copilot please review. Focus on: factual accuracy of the changelog inventory against the merged PR list (#23-#44, #47, #48, #56-#61), the wire-compat upgrade-note claims, and the Chart.yaml version/a...

Review finding: Chart.yaml now says appVersion: "v0.2.0", but values.yaml still defaults image.tag: dev, and templates/_helpers.tpl renders the image solely from .Values.image.tag. Thus the chart metadata names v0.2.0 while a default install deploys dart:dev/dart-k8s:dev; either bump the default tag (and its explanatory comment) or retain appVersion: "dev" until release images are available. I also cross-checked the listed PRs: the 30-number inventory and subsystem grouping are complete, and the direct-commit hashes/dates match GitHub. The wire-compat wording is consistent with the documented changes, subject to the stated malformed/duplicate-ID exclusions.

…e tag to deploy

Copilot review on PR #63: appVersion v0.2.0 while image.tag defaults to dev
read as incoherent. Keeping appVersion (chart metadata naming the DART
release the chart targets) and clarifying values.yaml instead: no registry
publishes dart images, the default dev tag is a local-build convention, and
operators deploying the release should set tag to the chart's appVersion.
@Coldwings

Copy link
Copy Markdown
Collaborator Author

@copilot addressed: kept appVersion v0.2.0 as chart metadata and clarified values.yaml — images are source-built (no registry publication), dev is the local-build default, and release deploys should set tag to the chart's appVersion. Please re-review.

Copilot AI commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

@copilot addressed: kept appVersion v0.2.0 as chart metadata and clarified values.yaml — images are source-built (no registry publication), dev is the local-build default, and release deploys should...

Re-review: the values.yaml clarification resolves the deployment/documentation mismatch—dev is explicitly the source-build default, and release users are told to set image.tag to v0.2.0. One minor stale comment remains in Chart.yaml: appVersion tracks the DART image tag the chart defaults to is no longer true because the chart defaults to image.tag: dev. Please change that comment to say it identifies the release image/tag the chart targets (or equivalent).

Copilot re-review on PR #63: the old comment claimed appVersion is the
chart's default image tag; the default stays dev. appVersion identifies
the DART release (image tag) this chart version targets.
@Coldwings

Copy link
Copy Markdown
Collaborator Author

@copilot fixed the stale Chart.yaml comment — appVersion now documented as naming the targeted release image tag, not the chart default. Please re-review.

Copilot AI commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

@copilot fixed the stale Chart.yaml comment — appVersion now documented as naming the targeted release image tag, not the chart default. Please re-review.

Re-review complete: the Chart.yaml comment now accurately distinguishes the targeted release image tag (appVersion) from the source-build default (image.tag: dev). The Helm metadata and accompanying values.yaml guidance are consistent; no further changes needed.

@Coldwings
Coldwings merged commit 0a91dc8 into main Aug 26, 2026
8 checks passed
@Coldwings
Coldwings deleted the release/v0.2.0 branch August 26, 2026 07:59
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.

Proposal: cut release v0.1.1 (28 fix PRs since v0.1.0, wire-compatible)

2 participants