Skip to content

docs: Fix typo and grammar in fix-documentation-issues SKILL.md - #6297

Open
manuth07 wants to merge 1 commit into
wso2:masterfrom
manuth07:fix/typo-in-github-skill-md
Open

docs: Fix typo and grammar in fix-documentation-issues SKILL.md#6297
manuth07 wants to merge 1 commit into
wso2:masterfrom
manuth07:fix/typo-in-github-skill-md

Conversation

@manuth07

@manuth07 manuth07 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Purpose

Fixed a spelling and grammatical error in .github/ISSUE_TEMPLATE/skills/fix-documentation-issues/SKILL.md.

  • Corrected "availables" to "available versions".
  • Improved sentence phrasing from can find the all version availables to can find all the available versions.

Related PRs

N/A

Test environment

N/A (Documentation fix)

Security checks

@CLAassistant

CLAassistant commented Aug 2, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The version-detection instruction now uses corrected wording for version directories under en/identity-server/.

Changes

Documentation wording correction

Layer / File(s) Summary
Correct version detection wording
.github/skills/fix-documentation-issues/SKILL.md
Corrected the wording that describes available version directories under en/identity-server/.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the documentation spelling and grammar fix.
Description check ✅ Passed The description includes all required sections and clearly explains the documentation changes and test scope.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Warning

⚠️ This pull request shows signs of AI-generated slop (description_diff_mismatch). It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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 @.github/skills/fix-documentation-issues/SKILL.md:
- Line 37: Update the wording in the documentation instruction to say “all
available versions” and add a space before the parenthesis following “issue.”
Preserve the surrounding instruction and meaning.
🪄 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: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: d7800dc6-2c2d-4ecd-8326-d0f687962d07

📥 Commits

Reviewing files that changed from the base of the PR and between f7512b5 and 597e286.

📒 Files selected for processing (1)
  • .github/skills/fix-documentation-issues/SKILL.md

1. Version detection:
- If the related Github issue explicitly mentions a version (in title/body/labels):
* FIRST: Fix that specific version mentioned in the issue(from "en/identity-server/" can find the all version availables)
* FIRST: Fix that specific version mentioned in the issue(from "en/identity-server/" can find the all version available versions)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Correct the remaining duplicated wording.

Line 37 still says all version available versions. Replace it with clear wording and add the missing space before the parenthesis.

Proposed fix
-     * FIRST: Fix that specific version mentioned in the issue(from "en/identity-server/" can find the all version available versions)
+     * FIRST: Fix that specific version mentioned in the issue (you can find all the available versions in "en/identity-server/")
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
* FIRST: Fix that specific version mentioned in the issue(from "en/identity-server/" can find the all version available versions)
* FIRST: Fix that specific version mentioned in the issue (you can find all the available versions in "en/identity-server/")
🤖 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 @.github/skills/fix-documentation-issues/SKILL.md at line 37, Update the
wording in the documentation instruction to say “all available versions” and add
a space before the parenthesis following “issue.” Preserve the surrounding
instruction and meaning.

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