docs: show core middleware import (#302) + drop $ from copyable shell blocks (#343)#416
Merged
Conversation
…ble shell blocks (#343) #302 — Core middleware pages used `middleware.X(...)` without ever showing where the `middleware` package comes from, which newcomers couldn't locate. Add a prominent import block right after each core middleware page's intro: import "github.com/labstack/echo/v5/middleware" Applied to all 20 core middleware pages across every locale (en/zh-cn/ja/es/ pt-br); the lead-in sentence is translated per locale, the import code block is identical everywhere. #343 — session.md showed curl examples with a leading `$ ` prompt inside the code block, so the copy button included the `$` and made pasting awkward. Drop the `$ ` prompt from the command lines (all 5 locales). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes two long-standing docs issues.
#302 — Missing middleware package import
Core middleware pages showed
middleware.CORS(...),middleware.BasicAuth(...), etc. but never indicated which packagemiddlewareis, so newcomers couldn't find the import. This adds a prominent import block right after each core middleware page's intro:#343 — Copy button included the
$promptsession.mdshowedcurlexamples with a leading$prompt inside the code block, so clicking copy grabbed the$and made pasting awkward. Removed the$prompt from the command lines (all 5 locales).Verification
curllines; no$prompts remain anywhere in the docs.Closes #302
Closes #343
🤖 Generated with Claude Code