preserve world pkg/openvpn as legacy reference - #40
Merged
Conversation
world repo deleted 2026-07-21 after full migration; this is the Go mini-CA that generated the VPN PKI — kept verbatim (final commit 929ac5a) for the 2030 cert renewal. Reference only: not compiled, not imported
![ben-s-pull-request-reviewer[bot]](https://avatars.githubusercontent.com/u/1193284?s=60&v=4){kind=small}
There was a problem hiding this comment.
Based on my review of PR #40 "preserve world pkg/openvpn as legacy reference":
Review Summary:
- All 7 files are new additions in
docs/legacy/world-pkg-openvpn/ - Files are preserved verbatim from
github.com/bborbe/worldfinal commit929ac5a - README accurately documents provenance, purpose (2030 cert renewal reference), and states clearly "Not compiled, not imported — reference only"
- Renewal gotchas correctly reference
bundles/openvpn/README.md - Verified no imports/references to these files from the main codebase
- LICENSE file exists in repo root
- The preserved Go files import
github.com/bborbe/worldpackages (deleted repo) — expected for verbatim preservation
{
"verdict": "approve",
"summary": "PR correctly preserves world/pkg/openvpn as reference-only documentation in docs/legacy/. README accurately describes source commit 929ac5a, purpose (2030 cert renewal), and explicitly marks files as non-compiled reference. All 7 files are verified unlinked from any build.",
"comments": [],
"concerns_addressed": [
"correctness: README accurately describes source commit 929ac5a and renewal gotchas - VERIFIED",
"correctness: All Go source files are preserved verbatim and unlinked from any build - VERIFIED"
]
}
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
1 participant
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.
The world repo was deleted today (2026-07-21) after the full migration completed (bw #36–#39 + world #19). This preserves the Go mini-CA / OpenVPN provisioning code verbatim (world final commit
929ac5a) underdocs/legacy/world-pkg-openvpn/— the authoritative reference for how the VPN PKI was generated, needed at the 2030 cert renewal.Reference only: plain files, not compiled, not imported. README documents provenance + pointers to the renewal gotchas in
bundles/openvpn/README.md.