feat: add brand assets and update README with asset details by shenxianpeng · Pull Request #91 · cpp-linter/.github · GitHub
Skip to content

feat: add brand assets and update README with asset details#91

Merged
shenxianpeng merged 2 commits into
mainfrom
feature/add-branch-folder
Jun 25, 2026
Merged

feat: add brand assets and update README with asset details#91
shenxianpeng merged 2 commits into
mainfrom
feature/add-branch-folder

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Jun 24, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Documentation
    • Expanded the branding guide with a clearer overview of available brand assets, including logos, wordmarks, banners, avatars, and social preview images.
    • Added guidance on where different assets are intended to be used, plus notes for editing source artwork and required font setup.

@2bndy5

2bndy5 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

@shenxianpeng

Copy link
Copy Markdown
Member Author

it looks intresting. I used https://claude.ai/design to genreate the assets in this PR

@shenxianpeng shenxianpeng marked this pull request as ready for review June 25, 2026 16:05
@shenxianpeng shenxianpeng merged commit 16feb47 into main Jun 25, 2026
5 of 6 checks passed
@shenxianpeng shenxianpeng deleted the feature/add-branch-folder branch June 25, 2026 16:06
@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

The README now documents the cpp-linter brand asset library, including sourced artwork and color palette, social preview filenames, org-level asset files, source artwork categories, and editing notes for GIMP and Abril Fatface.

Changes

Branding README

Layer / File(s) Summary
Brand asset documentation
branding/README.md
Adds brand origin and palette details, social preview image references, org-level asset listings, source artwork categories, and GIMP/font editing notes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding brand assets and updating the README with asset details.
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.

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


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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants