Repository navigation
Prepare the repository for open source - #143
Merged
Merged
Conversation
Remove PostHog analytics entirely: the telemetry package, its baked build keys, the ldflags and verify-bake checks, and the release workflow vars. The review client surface it resolved moves into the CLI, where it still labels gx Cloud review history. Remove the vendored review-knowledge corpus and its indexing pipeline. The documents are third-party material under their publishers' terms. The source list stays at docs/review-knowledge-sources.yaml, which the language-tag contract test now reads. Add NOTICE with copyright to Joe LaChance under AGPL-3.0-only, plus CONTRIBUTING, SECURITY, CODE_OF_CONDUCT, CODEOWNERS, issue and PR templates, and README sections on privacy, review knowledge, building from source, and licensing. Add a Go CI workflow on Linux and macOS. Delete the outdated CONTEXT.md and the unused docs-site stub, and scrub local /Users/joe paths from code, tests, and docs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> gx: https://gx.run/r/Yiv-PoOruzSUMknebVoCHQ
joelachance
force-pushed
the
claude/open-source-project-steps-9465ff
branch
from
October 6, 2026 20:08
0a83e44 to
e05be3a
Compare
TestCleanupStaleStacksMarksMissingBookmarkStacksClosed committed in a temp repo without setting user.name or user.email, so it borrowed the machine's identity. Linux CI runners have none, and git refused the commit. The test now configures the repo itself, as its sibling tests do. The full suite also passes with user.useConfigOnly set and no global identity. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> gx: https://gx.run/r/omjZcELdYNxGRD04BV_leQ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Gets the repo ready to announce as an open-source project.
Removed
--clientsurface still labels gx Cloud review history.gx enhance --clientis gone because it only fed analytics.docs/review-knowledge-sources.yaml./Users/joepath in code, tests, and docs.Added
ci.yml, which runs gofmt, vet, build, and test on Linux and macOS.Fixed
TestCleanupStaleStacksMarksMissingBookmarkStacksClosedrelied on the machine's git identity and failed on Linux runners. It now sets its own.Merge this before anything else. The
GX_POSTHOG_KEYandGX_POSTHOG_HOSTrepository variables are already deleted. Until this lands, the release workflow on main still expects them, and its bake check fails on the next push to main.Follow-ups after merge
🤖 Generated with Claude Code