Skip to content

docs(readme): show logo in both readmes#81

Merged
ymkiux merged 1 commit intomainfrom
docs/readme-logo
Apr 9, 2026
Merged

docs(readme): show logo in both readmes#81
ymkiux merged 1 commit intomainfrom
docs/readme-logo

Conversation

@awsl233777
Copy link
Copy Markdown
Collaborator

@awsl233777 awsl233777 commented Apr 9, 2026

Summary

  • show res/logo.png at the top of README.md
  • show the same logo at the top of README.en.md

Validation

  • git diff --check -- README.md README.en.md

Summary by CodeRabbit

  • Documentation
    • Added a centered logo image to the project README files.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 378e47ea-947f-4593-986d-996e3db2b034

📥 Commits

Reviewing files that changed from the base of the PR and between f141392 and 7da0ed6.

📒 Files selected for processing (2)
  • README.en.md
  • README.md
📜 Recent review details
🔇 Additional comments (2)
README.md (1)

3-4: Logo insertion is clean and accessible.

Placement is consistent with the centered header, the relative asset path is appropriate, and alt text is present.

README.en.md (1)

3-4: English README logo update also looks good.

The header stays consistent with README.md, and the image markup is clear and valid.


📝 Walkthrough

Walkthrough

Both README files (English and standard versions) were updated with a centered logo image positioned at the top, pointing to res/logo.png with a width of 180 pixels. No functional or structural documentation changes were made.

Changes

Cohort / File(s) Summary
Logo Addition
README.en.md, README.md
Added centered <img> tag referencing res/logo.png at the top of both README files with 180px width.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A logo hops into view,
Centered and proud, shiny and new,
At the top where all eyes shall roam,
CodeRabbit's welcome home! 🏠✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding a logo image to both README files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/readme-logo

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 and usage tips.

@ymkiux ymkiux merged commit 0f6ba66 into main Apr 9, 2026
7 checks passed
@ymkiux ymkiux deleted the docs/readme-logo branch April 9, 2026 11:27
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