Skip to content

docs: recommend proto's built-in ZLS toolchain - #9

Merged
konomae merged 1 commit into
mainfrom
codex/recommend-builtin-toolchain
Sep 5, 2026
Merged

docs: recommend proto's built-in ZLS toolchain#9
konomae merged 1 commit into
mainfrom
codex/recommend-builtin-toolchain

Conversation

@konomae

@konomae konomae commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Recommend using proto's built-in ZLS toolchain instead of this plugin
  • Link to the proto v0.62 announcement
  • Highlight the recommendation with a GitHub Markdown alert

Testing

  • Documentation-only change
  • Verified with git diff --check

Copilot AI lite review requested due to automatic review settings September 5, 2026 02:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The change is documentation-only, uses valid GitHub alert syntax, and cleanly communicates the new recommendation with an appropriate reference link.

Pull request overview

This PR updates the repository documentation to direct users to proto’s official built-in ZLS toolchain (introduced in proto v0.62), instead of relying on this WASM plugin.

Changes:

  • Adds a GitHub Markdown “IMPORTANT” alert near the top of the README.
  • Links to the proto v0.62 announcement post to support the recommendation.
File summaries
File Description
README.md Adds an IMPORTANT notice recommending proto’s built-in ZLS toolchain and links the proto v0.62 announcement.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@konomae
konomae merged commit 070a493 into main Sep 5, 2026
9 checks passed
@konomae
konomae deleted the codex/recommend-builtin-toolchain branch September 5, 2026 02:50
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