Skip to content

Removed brackets in the example of phoenix#1881

Closed
zyantw wants to merge 1 commit into
mainfrom
zyantw-patch-1
Closed

Removed brackets in the example of phoenix#1881
zyantw wants to merge 1 commit into
mainfrom
zyantw-patch-1

Conversation

@zyantw

@zyantw zyantw commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Preview: https://deploy-preview-1881--adk-docs-preview.netlify.app/integrations/phoenix/

Removing the brackets in this example that was triggering a mistake in the link check of the branch for my pull request: #1875

Removing the brackets in this example that was triggering a mistake in the link check of the branch for my pull request: #1875
@netlify

netlify Bot commented Jun 26, 2026

Copy link
Copy Markdown

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit 357c5a4
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/6a3efb470f6fce00081cbe5a
😎 Deploy Preview https://deploy-preview-1881--adk-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@zyantw zyantw requested a review from joefernandez June 26, 2026 22:21
@zyantw zyantw self-assigned this Jun 26, 2026
@koverholt

koverholt commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

The example link that this PR is attempting to fix is already handled/ignored by the .lycheeignore rule here:
https://github.com/google/adk-docs/blob/main/.lycheeignore#L13-L14.

This link validator CI check was actually failing because phoenix.arize.com went offline, I updated the two places that the failing link was used in 027855e as part of #1877. So if you sync your existing PRs with main now using the "Update branch" button, the CI check will pass.

@koverholt koverholt closed this Jun 26, 2026
@koverholt koverholt deleted the zyantw-patch-1 branch June 26, 2026 23:03
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