Skip to content

[Docs] Remove GitHub proxy mentions from active docs - #3540

Merged
adamziel merged 3 commits into
trunkfrom
codex/address-github-proxy-mentions
Apr 28, 2026
Merged

[Docs] Remove GitHub proxy mentions from active docs#3540
adamziel merged 3 commits into
trunkfrom
codex/address-github-proxy-mentions

Conversation

@adamziel

@adamziel adamziel commented Apr 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replace active docs mentions of the old GitHub repository proxy with git:directory guidance.
  • Clarify that url resources should point at built ZIP artifacts for plugins/themes that need a build step.
  • Keep the merged .git support guidance intact: repository URLs with or without a trailing .git suffix are supported.
  • Leave changelog history and Blueprint compatibility code/tests intact because they intentionally refer to legacy github-proxy.com migration behavior.

Validation

  • rg -n -i 'github-proxy|github proxy' packages/docs/site/docs --glob '!**/changelog.md' returned no active docs matches
  • git diff --check HEAD~1..HEAD
@adamziel
adamziel force-pushed the codex/address-github-proxy-mentions branch from dab1316 to 25532bf Compare April 28, 2026 09:55
@adamziel adamziel added [Type] Documentation Improvements or additions to documentation [Type] Enhancement New feature or request labels Apr 28, 2026
@adamziel
adamziel marked this pull request as ready for review April 28, 2026 10:04
@adamziel
adamziel requested review from a team, brandonpayton and Copilot April 28, 2026 10:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR updates the documentation to remove references to the legacy GitHub proxy flow, steering users toward git:directory for source-based installs and url for built ZIP artifacts.

Changes:

  • Replace active docs references to the GitHub proxy with git:directory guidance.
  • Clarify that build-step plugins/themes should be installed via a built ZIP artifact using url.
  • Update troubleshooting text to reflect the new recommended flows.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
packages/docs/site/docs/main/guides/github-action-pr-preview.md Updates troubleshooting guidance to direct users to git:directory/url instead of proxy ZIP services.
packages/docs/site/docs/main/guides/for-theme-developers.md Replaces GitHub proxy messaging with git:directory and built-artifact guidance.
packages/docs/site/docs/main/guides/for-plugin-developers.md Replaces GitHub proxy messaging with git:directory and built-artifact guidance (incl. Composer/npm builds).
packages/docs/site/docs/blueprints/04-resources.md Updates url resource docs to emphasize built ZIP artifacts and prefers git:directory for source.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/docs/site/docs/main/guides/github-action-pr-preview.md Outdated
Comment thread packages/docs/site/docs/main/guides/github-action-pr-preview.md Outdated
Comment thread packages/docs/site/docs/blueprints/04-resources.md Outdated
adamziel added a commit that referenced this pull request Apr 28, 2026
The auto-labeler skipped drafts (job-level `draft == false` check) and
only triggered on `opened`, so a PR opened as a draft never got labels —
even after being marked ready. PR #3540 is a recent example.

Two small changes:
- Drop the draft gate so drafts get labeled on open like everything
else.
- Add `ready_for_review` to the trigger types as a backstop, in case a
PR somehow slips through the `opened` event.
@adamziel
adamziel force-pushed the codex/address-github-proxy-mentions branch from f081788 to c6abd89 Compare April 28, 2026 13:38
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@adamziel adamziel changed the title [codex] Remove GitHub proxy mentions from active docs Apr 28, 2026
…-proxy-mentions

# Conflicts:
#	packages/docs/site/docs/main/guides/github-action-pr-preview.md
@adamziel
adamziel merged commit 3637da8 into trunk Apr 28, 2026
47 checks passed
@adamziel
adamziel deleted the codex/address-github-proxy-mentions branch April 28, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Documentation Improvements or additions to documentation [Type] Enhancement New feature or request

2 participants