-
Notifications
You must be signed in to change notification settings - Fork 76
docs: fix typos, code syntax issues, and improve Prisma AIRS documentation #1012
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
- Fix spelling: "prerequsities" → "prerequisites" in usecases.md - Fix spelling: "adminstration" → "administration" in airuntimesecurityapi.md - Fix spelling: "intecept" → "intercept" in airuntimesecurityapi.md - Fix security issue: HTTP → HTTPS for Strata Cloud Manager login URL - Fix grammar: add missing "to" in Custom Topics bullet point These corrections improve documentation quality and fix a security concern where the Strata Cloud Manager login link used HTTP instead of HTTPS.
Fixed several issues found while reviewing the Prisma AIRS documentation: Spelling corrections: - "Synchrounous" → "Synchronous" in errorcodes.md - "Asycio" → "Asyncio" in pythonsdkasynciousage.md heading - "storty" → "story" in code example - "structur" → "structure" in incomplete comment - "Asyncronous" → "Asynchronous" in pythonsdkusage.md Python code fixes: - Remove incorrect 'await' keyword in synchronous scanner.close() calls (pythonsdkusage.md lines 58, 183, 221, 331) - These examples are synchronous, so await causes SyntaxError Documentation clarity: - Fix circular reference: "Review the scan API endpoints in the scan API endpoints" now links to actual endpoints section - Add missing closing quote in curl command (usecases.md line 122) - Remove Python-style # comments from JSON examples (invalid JSON syntax) Terminology consistency: - Standardize "AI Network" → "AI Runtime" in errorcodes.md - Fix markdown lint: wrap bare URLs in angle brackets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have several places in this file where the edit changed 'structure' to 'structuree'. Please fix this.
- Fix spelling: "structuree" → "structure" in pythonsdkasynciousage.md (appears in two code comment locations)
|
Visit the preview URL for this PR (updated for commit 8f24c5f): https://pan-dev-f1b58--pr1012-456e83yl.web.app (expires Tue, 18 Nov 2025 03:51:37 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 9359a9fa100d5fdb082f75280473a90b1f96eb51 |
Summary
This PR improves the Prisma AIRS documentation quality through typo corrections, code syntax fixes, and consistency improvements.
Changes Made
Commit 1: Initial Documentation Fixes
Spelling Corrections:
Security Fix:
4. airuntimesecurityapi.md (line 53): Changed Strata Cloud Manager login URL from HTTP to HTTPS
http://stratacloudmanager.paloaltonetworks.com/https://stratacloudmanager.paloaltonetworks.com/Grammar Fix:
5. airuntimesecurityapi.md (line 59): Added missing word "to" in Custom Topics bullet
Commit 2: Code Syntax and Additional Documentation Fixes
Additional Spelling Corrections:
Critical Python Code Fixes:
awaitkeyword in synchronousscanner.close()callsawaitoutside async function causes SyntaxErrorawait scanner.close()toscanner.close()Documentation Clarity:
#comments from JSON examples (JSON doesn't support comments)Consistency Improvements:
Files Modified
products/prisma-airs/api/airuntimesecurity/airuntimesecurityapi.mdproducts/prisma-airs/api/airuntimesecurity/usecases.mdproducts/prisma-airs/api/airuntimesecurity/errorcodes.mdproducts/prisma-airs/api/airuntimesecurity/pythonsdkusage.mdproducts/prisma-airs/api/airuntimesecurity/pythonsdkasynciousage.mdTesting
All changes are documentation-only and improve quality, security, and code accuracy.