Skip to content

Fix grammar issues in guardrail documentation #1260

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mshsheikh
Copy link
Contributor

@mshsheikh mshsheikh commented Jul 26, 2025

✅ The article usage is now grammatically correct with "an" before class names starting with vowel sounds
✅ The possessive form correctly uses "function's name" instead of "function name"
✅ The redundant word "do" has been removed from "to do check" → "to check"
✅ The line length violations (E501) have been resolved with proper docstring formatting
✅ Overall documentation now maintains professional consistency and grammatical accuracy

1. ✅ Fixed article usage ("a" to "an") before class names starting with vowel sounds
2. ✅ Corrected possessive form ("function name" to "function's name")
3. ✅ Removed redundant word ("do" in "to do check" → "to check")
Corrected multiple documentation issues across the guardrail implementation:
- Fixed article usage ("a" → "an") before class names starting with vowel sounds
- Corrected possessive forms ("function name" → "function's name")
- Removed redundant words ("to do check" → "to check")
- Resolved line length violations (E501) by restructuring long docstring lines
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.

1 participant