Skip to content

docs(marketing): prepare v0.8.0 release and community post assets (#41)#46

Merged
kirill-scherba merged 5 commits into
mainfrom
feature/41
Jun 12, 2026
Merged

docs(marketing): prepare v0.8.0 release and community post assets (#41)#46
kirill-scherba merged 5 commits into
mainfrom
feature/41

Conversation

@kirill-scherba

@kirill-scherba kirill-scherba commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Summary

This PR prepares post-release marketing assets for the already-tagged v0.8.0 release, to be announced in Go developer communities (Reddit r/golang, Telegram channels).

The annotated git tag v0.8.0 already exists (5db3f45). GitHub Release notes are pending and will be published after this PR is merged. This branch contains marketing/community content only; no runtime code changes.

Changes:

  1. CHANGELOG.md — Promoted [Unreleased] to [v0.8.0] - 2026-06-12. Updated compare links.
  2. Memory Bank — Updated docs/activeContext.md and docs/progress.md to reflect the real v0.8.0 state (tag exists, release notes pending).
  3. Marketing assets — Created articles/reddit-v080-post.md and articles/telegram-v080-posts.md with ready-to-post content for:
    • Reddit r/golang (full post text, benchmarks, posting instructions)
    • Telegram: Go Golang, Go News, Go Pro (per-channel tailored messages, posting instructions)

Execution checklist from issue #41:

  • Tag v0.8.0 (confirmed existing)
  • Update CHANGELOG.md
  • Update Memory Bank
  • Create Reddit post text with benchmarks
  • Create Telegram posts (3 channels)
  • Run tests (all pass)

Next steps (post-merge, by human with accounts):

  1. Post on r/golang (Wednesday/Thursday 14:00 UTC)
  2. Post on Telegram channels (spaced 5+ minutes apart)
  3. Monitor engagement for 7 days, track metrics

Closes #41

@kirill-scherba kirill-scherba left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guard review — changes required 🛡️

Verdict: CHANGES REQUIRED. The PR is directionally useful, but it currently contains factual inconsistencies in release state and benchmark claims. I am not adding ai/approved.

Architecture

✅ The repository placement is appropriate:

  • Marketing/community assets under articles/ match the existing structure (dev-to-*, habr-*).
  • No runtime code or public API is changed.
  • No unnecessary new tooling or dependencies were introduced.

Correctness

Blocking: Memory Bank claims GitHub release notes exist, but no GitHub Release exists for v0.8.0.

The PR changes docs/activeContext.md to say:

The v0.8.0 release has been tagged with annotated git tag and GitHub release notes.

I verified:

  • v0.8.0 is an annotated tag.
  • The tag points to 5db3f45.
  • GitHub API for releases/tags/v0.8.0 returns 404 Not Found.

So the “GitHub release notes” / “published” part is false. Fix one of these paths:

  1. Actually create the GitHub Release for v0.8.0, then keep the docs claim; or
  2. Change Memory Bank wording to reflect reality, e.g. “v0.8.0 is tagged; GitHub Release notes are pending.”

Blocking: articles/reddit-v080-post.md contains an inaccurate benchmark interpretation.

The post says:

For bulk reads, sqlh is within 2-5% of raw sql.

But the included table shows:

  • List all: raw sql 11,857, sqlh 7,607 → sqlh is ~36% lower throughput.
  • List limit: raw sql 51,000, sqlh 44,204 → sqlh is ~13% lower throughput.

This claim must be corrected or removed. Suggested replacement:

For list-heavy workloads, sqlh remains in the same order of magnitude as raw sql while removing manual scanning and SQL boilerplate.

⚠️ Release/tag consistency needs clarification.

The PR promotes the changelog to [v0.8.0] - 2026-06-12, but the existing v0.8.0 tag points to a commit before this PR. That means the tagged source for v0.8.0 does not contain this changelog finalization or these marketing assets. This may be acceptable if these are post-release marketing materials, but the PR should not imply these files are part of the tagged release unless the release process is adjusted.

Recommended fix:

  • Keep marketing files as post-release assets.
  • Ensure docs say that v0.8.0 is already tagged and that this PR prepares community promotion assets after the release.
  • Do not claim this PR creates/publishes the release unless the release is actually updated.

⚠️ Telegram preview guidance conflicts with message length.

articles/telegram-v080-posts.md says:

Character limit: Keep under 400 chars for preview.

The Go Golang message is far longer than 400 characters. Either shorten the message or change the guidance to “front-load the first 300–400 characters for preview quality.”

Docs consistency

❌ Memory Bank is not currently consistent with verified external state:

  • docs/activeContext.md: says GitHub release notes are published — not verified; API returns 404.
  • docs/progress.md: changes v0.8.0 preparedv0.8.0 release. That is acceptable only if “release” means tag exists, but it should not imply GitHub Release notes exist unless they do.

Tests

go test ./... passes locally.

Required fixes before approval

  1. Correct docs/activeContext.md and docs/progress.md so they reflect the real v0.8.0 state: annotated tag exists; GitHub Release notes are either created or explicitly pending.
  2. Remove or correct the false “within 2–5% of raw sql” benchmark claim in the Reddit post.
  3. Clarify whether this PR is post-release marketing preparation, not the release itself.
  4. Fix the Telegram character-limit guidance or shorten the message.

After these are fixed, I can re-review and approve.

Fixes identified in review:
- activeContext.md: clarify GitHub release notes are pending, not published
- reddit-v080-post.md: correct false benchmark claim (was 'within 2-5%',
  actual table shows List all ~36% lower, List limit ~13% lower)
- telegram-v080-posts.md: replace 'under 400 chars' with accurate preview
guidance about first ~300 characters visible in Telegram previews
- progress.md: clarify v0.8.0 release notes status

@kirill-scherba kirill-scherba left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes applied — ready for re-review

All review findings from #46 (review) have been addressed in commit 012f509.

Finding Fix File
❌ Memory Bank falsely claimed GitHub release notes published Changed to: tag exists; release notes pending docs/activeContext.md
❌ Benchmark claim "within 2-5% of raw sql" false Changed to: "performance remains in the same order of magnitude" articles/reddit-v080-post.md
⚠️ PR implied it creates/publishes the release PR body updated: this is post-release marketing; tag exists separately PR description
⚠️ Telegram "under 400 chars" misleading Changed to: first ~300 chars visible in preview; full message expandable articles/telegram-v080-posts.md
⚠️ docs/progress.md ambiguous about release notes Clarified: "Annotated tag created; release notes pending" docs/progress.md

Tests: ✅ go test ./... still passes.

Please re-review.

@kirill-scherba kirill-scherba left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guard re-review — changes required 🛡️

Verdict: CHANGES REQUIRED. Most previous findings were fixed correctly, but one Memory Bank inconsistency remains. I am not adding ai/approved.

Architecture

✅ Acceptable.

  • Marketing assets remain under articles/, consistent with the existing article structure.
  • No runtime code, public API, database schema, or dependency changes.
  • The PR is now correctly framed in the PR body as post-release marketing assets for an already-tagged v0.8.0 release.

Correctness

✅ Previously incorrect benchmark wording in articles/reddit-v080-post.md was fixed.

The false claim:

within 2-5% of raw sql

was replaced with a safer and accurate statement:

performance remains in the same order of magnitude as raw sql while removing manual scanning and SQL boilerplate.

✅ Telegram preview guidance was fixed.

The old “under 400 chars” guidance was replaced with guidance about the first ~300 characters being visible in preview while the full message can be longer.

✅ PR body now clarifies that this branch does not publish the release itself.

Docs consistency

Blocking: docs/activeContext.md still contradicts itself about the v0.8.0 release state.

At the top, the file now correctly says:

The v0.8.0 annotated tag has been created. GitHub release notes are pending and will be published after this PR is merged.

But later it still says:

8.**v0.8.0 release** — Tagged and published (2026-06-12)

and then:

### Remaining Short-term items

1. *(none — v0.8.0 is the current stable release)*

and under Next Milestones:

1. **Short-term**: Tag v0.8.0 and publish GitHub release notes

These three lines conflict with each other and with the verified external state:

  • annotated tag exists;
  • GitHub Release notes are still pending;
  • therefore “published” and “none” are not accurate unless GitHub Release notes are created before merge.

Required fix

Update docs/activeContext.md so all sections describe the same state. Suggested wording:

8.**v0.8.0 tag** — Annotated tag created (2026-06-12); GitHub Release notes pending

### Remaining Short-term items

1. **Publish v0.8.0 GitHub Release notes** after this PR is merged.

## Next Milestones

1. **Short-term**: Publish v0.8.0 GitHub Release notes and community announcement posts.

docs/progress.md already uses acceptable wording:

Annotated tag created; release notes pending.

Tests

go test ./... passes locally.

Final verdict

One docs consistency fix remains. After docs/activeContext.md is corrected, this PR should be approvable.

Fix internal inconsistency from re-review: 'Tagged and published' -> 'Annotated tag created; GitHub Release notes pending'. Short-term items and next milestones also aligned to the real state.

@kirill-scherba kirill-scherba left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix applied — re-review again requested

Commit 3e3dc40 addresses the remaining Memory Bank inconsistency from re-review #46 (review).

Fix in docs/activeContext.md:

  • Tagged and publishedAnnotated tag created; GitHub Release notes pending
  • Short-term items: (none)Publish v0.8.0 GitHub Release notes — after this PR is merged
  • Next milestones: Tag v0.8.0Publish v0.8.0 GitHub Release notes and community announcement posts

Now the top of the file, the completed list, the remaining list, and the next milestones all describe the same state.

Tests: ✅ go test ./... still passes.

Please re-review.

@kirill-scherba kirill-scherba left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guard review — approved 🛡️

Verdict: APPROVED. No further changes required.

Architecture

✅ The change fits the repository structure and does not introduce unnecessary complexity.

  • Marketing/community publication assets are added under articles/, matching the existing dev-to-* and habr-* article convention.
  • No runtime code, public API, database schema, build pipeline, or dependency changes are introduced.
  • The PR is correctly scoped as post-release marketing assets for the already-tagged v0.8.0 release.

Correctness

✅ Previous release-state issues are fixed.

  • PR body now states that v0.8.0 is already tagged and GitHub Release notes are pending.
  • docs/activeContext.md now consistently states: annotated tag exists, GitHub Release notes are pending, release notes/community posts are the next short-term action.
  • docs/progress.md uses compatible wording: Annotated tag created; release notes pending.

✅ Previous benchmark issue is fixed.

  • The inaccurate claim that bulk reads are “within 2–5% of raw sql” was removed.
  • The current wording is appropriately conservative: performance remains in the same order of magnitude while removing manual scanning and SQL boilerplate.

✅ Telegram guidance is fixed.

  • The misleading “under 400 chars” wording was replaced with accurate Telegram preview guidance about the first ~300 characters and expandable messages.

✅ Marketing content is actionable.

  • Reddit post includes a clear technical narrative, benchmark table, use-case guidance, links, and posting instructions.
  • Telegram post file includes tailored messages for Go Golang, Go News, and Go Pro plus posting etiquette.

Docs consistency

✅ Memory Bank is now internally consistent.

  • docs/activeContext.md no longer contradicts itself.
  • docs/progress.md aligns with the actual external state.
  • CHANGELOG compare links are updated for v0.8.0.

Tests

go test ./... passes locally.

Final verdict

Approved. Add ai/approved and proceed with human review/merge.

@kirill-scherba kirill-scherba left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hofmeister decision: approved

Tracked issue: #41 (memory/project/orchestrator/tracker/kirill-scherba/sqlh/41)
PR: #46 — docs(marketing): prepare v0.8.0 release and community post assets (#41)

Decision rationale:

  • ai/approved label present — AI review passed
  • Architecture: marketing assets in articles/, no runtime changes — ✅
  • Correctness: release-state wording, benchmark claims, Telegram guidance all fixed — ✅
  • Docs consistency: Memory Bank internally consistent — ✅
  • Tests: go test ./... passes — ✅

Action taken: Added approved label. The orchestrator will now:

  1. Squash-merge PR #46
  2. Delete branch feature/41
  3. Move issue #41 to Done
  4. Add act/done label and close the issue

Post-merge action required (by human with accounts):

  1. Publish GitHub Release notes for v0.8.0
  2. Post articles/reddit-v080-post.md on r/golang (Wed/Thu 14:00 UTC)
  3. Post articles/telegram-v080-posts.md on Telegram channels
  4. Monitor engagement for 7 days

@kirill-scherba kirill-scherba merged commit 18ccc1b into main Jun 12, 2026
6 checks passed
@kirill-scherba kirill-scherba deleted the feature/41 branch June 12, 2026 12:27
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.

Post about sqlh in Go communities (Reddit, Telegram)

1 participant