Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates README installation instructions for Arch Linux to specify the correct pacman package name and add an alternative installation method using yay. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
📝 WalkthroughWalkthroughUpdated the README.md installation documentation to clarify the Arch Linux pacman command and added an alternative yay installation method for AUR users, improving the package installation instructions. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 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)
✨ Simplify code
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 |
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 22: Remove the leading shell prompt from the inline command examples to
satisfy markdownlint MD014: locate the occurrences of "$ sudo pacman -S
cachefetch-bin" (and the similar line at the other occurrence) in README.md and
replace them with the plain command "sudo pacman -S cachefetch-bin" (i.e.,
remove the "$ " prefix) so the code blocks contain just the command text for
easier copy/paste.
- Around line 41-45: The "yay -S cachefetch" command is an AUR install path but
is placed under the "From Source" section; update README.md by moving that code
block out of the "From Source" subsection or relabeling it (e.g., add a new
subsection titled "Arch Linux / AUR" or change the heading to "From AUR (Arch
Linux)") and update the accompanying text to make clear this uses an AUR helper
(`yay -S cachefetch`) rather than building from source.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
|
|
||
| ```sh | ||
| $ sudo pacman -S | ||
| $ sudo pacman -S cachefetch-bin |
There was a problem hiding this comment.
Remove shell prompts from command blocks (markdownlint MD014).
Line 22 and Line 44 include $ prompts even though no command output is shown, which triggers MD014 and reduces copy/paste friendliness.
Suggested diff
- $ sudo pacman -S cachefetch-bin
+ sudo pacman -S cachefetch-bin
...
- $ yay -S cachefetch
+ yay -S cachefetchAlso applies to: 44-44
🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 22-22: Dollar signs used before commands without showing output
(MD014, commands-show-output)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 22, Remove the leading shell prompt from the inline
command examples to satisfy markdownlint MD014: locate the occurrences of "$
sudo pacman -S cachefetch-bin" (and the similar line at the other occurrence) in
README.md and replace them with the plain command "sudo pacman -S
cachefetch-bin" (i.e., remove the "$ " prefix) so the code blocks contain just
the command text for easier copy/paste.
Comment on lines
+41
to
+45
| Or with `yay` | ||
|
|
||
| ```bash | ||
| $ yay -S cachefetch | ||
| ``` |
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial
Clarify that this is an AUR install path, not “From Source.”
This block appears under “From Source” but uses yay (AUR flow). Please relabel/move it to avoid misleading users.
Suggested wording
-Or with `yay`
+Or install from AUR with `yay`:🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 44-44: Dollar signs used before commands without showing output
(MD014, commands-show-output)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 41 - 45, The "yay -S cachefetch" command is an AUR
install path but is placed under the "From Source" section; update README.md by
moving that code block out of the "From Source" subsection or relabeling it
(e.g., add a new subsection titled "Arch Linux / AUR" or change the heading to
"From AUR (Arch Linux)") and update the accompanying text to make clear this
uses an AUR helper (`yay -S cachefetch`) rather than building from source.
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.
Summary by Sourcery
Documentation:
Summary by CodeRabbit