Skip to content

Remove internal references from public text - #192

Merged
askalf merged 1 commit into
mainfrom
scrub-internal-refs
Nov 1, 2026
Merged

askalf merged 1 commit into
mainfrom
scrub-internal-refs

Conversation

@askalf

@askalf askalf commented Nov 1, 2026

Copy link
Copy Markdown
Owner

The deploy runbooks and the compose, tunnel and env comments now use placeholders for the host, SSH key, tunnel and account instead of real names and addresses, and they describe the shared network and VPN in neutral terms. Workflow comments refer to the dispatcher by its role, and one changelog entry no longer prints a VPN exit address. Only comments and docs change; tests and the generated settings check pass.

Deploy docs, compose and tunnel comments now use placeholders for the host, key, tunnel and account instead of real names and addresses, and describe the shared network and VPN neutrally. Workflow comments name the dispatcher by role, and a changelog entry drops a VPN exit address.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying amnesia-site with  Cloudflare Pages  Cloudflare Pages

Latest commit: 99a5bd1
Status: ✅  Deploy successful!
Preview URL: https://06274150.amnesia-site.pages.dev
Branch Preview URL: https://scrub-internal-refs.amnesia-site.pages.dev

View logs

@github-actions github-actions Bot added documentation Documentation improvements github_actions Pull requests that update GitHub Actions code worker Cloudflare worker infra Infrastructure size/M 50-199 hand-written lines labels Nov 1, 2026

@sprayberry-redline sprayberry-redline left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Verdict: approve. The documentation and comment edits preserve the existing deployment topology and runtime behavior while replacing operator-specific details with placeholders. The proxy-settings template and its checked-in output use matching comment text, and workflow execution settings remain unchanged.

@askalf
askalf merged commit f22495e into main Nov 1, 2026
16 checks passed
@askalf
askalf deleted the scrub-internal-refs branch November 1, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Documentation improvements github_actions Pull requests that update GitHub Actions code infra Infrastructure size/M 50-199 hand-written lines worker Cloudflare worker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants