Skip to content

Fix reference update workflow#6

Merged
danbarr merged 1 commit into
mainfrom
fix-workflow
Jun 9, 2025
Merged

Fix reference update workflow#6
danbarr merged 1 commit into
mainfrom
fix-workflow

Conversation

@danbarr
Copy link
Copy Markdown
Collaborator

@danbarr danbarr commented Jun 9, 2025

The create-pull-request action expects to be on the base branch

- create-pull-request action expects to be on the base branch
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 9, 2025 1:00pm

@danbarr danbarr merged commit cbd5760 into main Jun 9, 2025
4 of 6 checks passed
@danbarr danbarr deleted the fix-workflow branch June 9, 2025 13:02
ChrisJBurns added a commit that referenced this pull request Apr 13, 2026
- Fix backend-discovery.mdx "Backend conversion" to say endpoint URL
  instead of service URL, since MCPServerEntry uses remoteURL (#1)
- Link MCPExternalAuthConfig to token-exchange-k8s.mdx instead of
  embedded auth server section (#2)
- Clarify complete example wording re: CA bundle ConfigMap (#3)
- Remove resource table links in intro.mdx per reviewer preference (#4)
- Add Telemetry row to comparison table (#5)
- Fix step numbering in "What's happening?" admonition (#6, #7)
- Fix TLS and header code blocks to use highlight comments (#8, #9)
- Fix broken duplicate header code block from bad merge

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
danbarr added a commit that referenced this pull request Apr 13, 2026
* Add dedicated Operator guide page for MCPServerEntry

Closes #688

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Address review feedback

- Add MCPServerEntry to intro.mdx opening paragraph (#1)
- Link all four resource types in intro.mdx table (#2)
- Include MCPServerEntry in discovered-to-inline RBAC note (#3)
- Move MCPRemoteProxy link from Next steps to Related info (#4)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Update docs/toolhive/guides-k8s/intro.mdx

Co-authored-by: Dan Barr <danbarr@users.noreply.github.com>

* Update docs/toolhive/guides-k8s/mcp-server-entry.mdx

Co-authored-by: Dan Barr <danbarr@users.noreply.github.com>

* Update docs/toolhive/guides-k8s/mcp-server-entry.mdx

Co-authored-by: Dan Barr <danbarr@users.noreply.github.com>

* Update docs/toolhive/guides-k8s/mcp-server-entry.mdx

Co-authored-by: Dan Barr <danbarr@users.noreply.github.com>

* Update docs/toolhive/guides-k8s/mcp-server-entry.mdx

Co-authored-by: Dan Barr <danbarr@users.noreply.github.com>

* Address PR review comments

- Fix backend-discovery.mdx "Backend conversion" to say endpoint URL
  instead of service URL, since MCPServerEntry uses remoteURL (#1)
- Link MCPExternalAuthConfig to token-exchange-k8s.mdx instead of
  embedded auth server section (#2)
- Clarify complete example wording re: CA bundle ConfigMap (#3)
- Remove resource table links in intro.mdx per reviewer preference (#4)
- Add Telemetry row to comparison table (#5)
- Fix step numbering in "What's happening?" admonition (#6, #7)
- Fix TLS and header code blocks to use highlight comments (#8, #9)
- Fix broken duplicate header code block from bad merge

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Dan Barr <danbarr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants