Skip to content

fix(docs): fix broken custom_functions link in v0.10 README#1259

Open
ppazosp wants to merge 1 commit intogoogle:mainfrom
ppazosp:fix/v010-custom-functions-link-v2
Open

fix(docs): fix broken custom_functions link in v0.10 README#1259
ppazosp wants to merge 1 commit intogoogle:mainfrom
ppazosp:fix/v010-custom-functions-link-v2

Conversation

@ppazosp
Copy link
Copy Markdown
Contributor

@ppazosp ppazosp commented Apr 22, 2026

Supersedes #1026, which was closed after I accidentally deleted my fork. Same change, rebased on current main.

Description

The v0.10 README links to docs/custom_functions.md which doesn't exist. The actual file is docs/a2ui_custom_functions.md.

Pre-launch Checklist

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation link for Custom Functions in the README.md file. The review feedback indicates that the pre-review checklist in the PR description is incomplete, which violates the repository's style guide requiring all checklist items to be completed.


- [Protocol](docs/a2ui_protocol.md)
- [Custom Functions](docs/custom_functions.md)
- [Custom Functions](docs/a2ui_custom_functions.md)
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.

medium

The Pre-Review Checklist in the PR description is incomplete, as the CHANGELOG and tests items are unchecked. This is a violation of the Repository Style Guide (lines 18-20), which requires all checklist steps to be completed.

References
  1. The Repository Style Guide requires that the Pre-Review Checklist in the PR description be fully completed. (link)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant