Update build-gateway-with-custom-policies.md.#350
Conversation
ap --version does not work ap version is the correct command.
📝 WalkthroughUpdated the macOS/Linux AP CLI installation instructions to use the correct WalkthroughThe macOS/Linux “Verify the installation” step in the AP CLI documentation now uses 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@en/docs/cloud/api-platform-gateway/build-gateway-with-custom-policies.md`:
- Line 56: Update the version-check command in
build-gateway-with-custom-policies.md to use the same syntax as the matching
guides, replacing ap version with ap --version unless ap version is explicitly
the intended command; if retaining ap version, update all remaining
version-check occurrences consistently.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 98240500-b382-40af-83c3-2cbfffe3a685
📒 Files selected for processing (1)
en/docs/cloud/api-platform-gateway/build-gateway-with-custom-policies.md
|
|
||
| ```bash | ||
| ap --version | ||
| ap version |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
set -euo pipefail
printf '\n## Target file\n'
sed -n '45,70p' en/docs/cloud/api-platform-gateway/build-gateway-with-custom-policies.md
printf '\n## Related guide\n'
sed -n '1,120p' en/docs/api-gateway/next/policies/custom-policies/building-gateway-with-custom-policies.md
printf '\n## Search for version command variants\n'
rg -n --no-heading -S 'ap --version|ap version' en/docs | sed -n '1,200p'Repository: wso2/docs-api-platform
Length of output: 5103
🏁 Script executed:
set -euo pipefail
printf '\nTarget file excerpt:\n'
nl -ba en/docs/cloud/api-platform-gateway/build-gateway-with-custom-policies.md | sed -n '50,62p'
printf '\nRelated guide excerpt:\n'
nl -ba en/docs/api-gateway/next/policies/custom-policies/building-gateway-with-custom-policies.md | sed -n '50,70p'
printf '\nAll occurrences:\n'
rg -n --no-heading -S 'ap --version|ap version' en/docsRepository: wso2/docs-api-platform
Length of output: 222
Keep the version-check command consistent. This page mixes ap version and ap --version, and the matching guides still use ap --version. If ap version is the intended syntax, update the remaining occurrences as well.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@en/docs/cloud/api-platform-gateway/build-gateway-with-custom-policies.md` at
line 56, Update the version-check command in
build-gateway-with-custom-policies.md to use the same syntax as the matching
guides, replacing ap version with ap --version unless ap version is explicitly
the intended command; if retaining ap version, update all remaining
version-check occurrences consistently.
ap --version does not work
ap version is the correct command.
Purpose
Checklist
llms.txt(located aten/docs/llms.txt) is updated for AI readiness content.Goals
Approach
User stories
Release note
Documentation
Training
Certification
Marketing
Automation tests
Security checks
Samples
Related PRs
Migrations (if applicable)
Test environment
Learning