[fix]: Minor tweak NixOS_small logo#2357
Conversation
Up to standards ✅🟢 Issues
|
There was a problem hiding this comment.
Pull Request Overview
The review confirms that the PR is up to standards with no logic, security, or quality issues identified. No blocking issues were found. Acceptance criteria regarding documentation formatting and asset styling appear addressed, though verification is recommended for the logo rendering and link correctness as these are not covered by automated tests.
Test suggestions
- Verify the NixOS_small ASCII logo renders with color index $3 on the right side instead of $4.
- Verify the README 'Build from source' link is correctly formatted and points to the Building wiki page.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify the NixOS_small ASCII logo renders with color index $3 on the right side instead of $4.
2. Verify the README 'Build from source' link is correctly formatted and points to the Building wiki page.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
| ## Build from source | ||
|
|
||
| See the Wiki: https://github.com/fastfetch-cli/fastfetch/wiki/Building | ||
| [See the Wiki](https://github.com/fastfetch-cli/fastfetch/wiki/Building) |
There was a problem hiding this comment.
Please revert unrelated changes
There was a problem hiding this comment.
Have reverted it now 👍
This reverts commit 72f4d11. "Remove unrelated changes"
Summary
Related issue (required for new logos for new distros)
Closes #
Changes
Screenshots
Checklist