Docs: Update PHP_CodeSniffer repository link#11644
Open
rodrigoprimo wants to merge 1 commit intoappwrite:1.9.xfrom
Open
Docs: Update PHP_CodeSniffer repository link#11644rodrigoprimo wants to merge 1 commit intoappwrite:1.9.xfrom
rodrigoprimo wants to merge 1 commit intoappwrite:1.9.xfrom
Conversation
Contributor
📝 WalkthroughWalkthroughThe pull request updates a documentation link in Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Contributor
Greptile SummaryThis PR updates a single documentation link in
Confidence Score: 5/5
Important Files Changed
Reviews (1): Last reviewed commit: "Docs: Update PHP_CodeSniffer repository ..." | Re-trigger Greptile |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
CONTRIBUTING.md (1)
92-92: Polish phrasing for clarity in the updated sentence.The link update is correct, but this sentence reads awkwardly (“pass in additional…”, “More lists…”). Consider a small wording cleanup.
✏️ Suggested wording
-This will give you a list of errors to rectify. If you need more information on the errors, you can pass in additional command line arguments to get more verbose information. More lists of available arguments can be found [on PHP_Codesniffer usage Wiki](https://github.com/PHPCSStandards/PHP_CodeSniffer/wiki/Usage). A very useful command line argument is `--report=diff`. This will give you the expected changes by the linter for easy fixing of formatting issues. +This will give you a list of errors to fix. If you need more details, you can pass additional command-line arguments for verbose output. A full list of available arguments is available on the [PHP_CodeSniffer usage wiki](https://github.com/PHPCSStandards/PHP_CodeSniffer/wiki/Usage). A very useful argument is `--report=diff`, which shows the expected linter changes for easier formatting fixes.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@CONTRIBUTING.md` at line 92, The sentence in CONTRIBUTING.md containing phrases "pass in additional command line arguments" and "More lists of available arguments" reads awkwardly; reword it to be clearer and more concise by replacing "pass in additional command line arguments" with "provide additional command-line arguments" (or similar), change "More lists of available arguments can be found" to "A full list of available arguments is on", and combine into one smooth sentence that still mentions the useful example flag `--report=diff` and the PHPCS usage wiki link.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@CONTRIBUTING.md`:
- Line 92: The sentence in CONTRIBUTING.md containing phrases "pass in
additional command line arguments" and "More lists of available arguments" reads
awkwardly; reword it to be clearer and more concise by replacing "pass in
additional command line arguments" with "provide additional command-line
arguments" (or similar), change "More lists of available arguments can be found"
to "A full list of available arguments is on", and combine into one smooth
sentence that still mentions the useful example flag `--report=diff` and the
PHPCS usage wiki link.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 593d9899-a026-4b2a-84c7-495996382a9b
📒 Files selected for processing (1)
CONTRIBUTING.md
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
1 participant
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.
PHP_CodeSniffer has moved to a new GitHub organization. The
squizlabs/PHP_CodeSnifferrepository is now archived and the project is actively maintained at https://github.com/PHPCSStandards/PHP_CodeSniffer.See: squizlabs/PHP_CodeSniffer#3932