Skip to content

doc: Add Fulcrum Genomics branding to README#33

Merged
nh13 merged 1 commit intomainfrom
doc/nh_update-fulcrum-logo
Apr 9, 2026
Merged

doc: Add Fulcrum Genomics branding to README#33
nh13 merged 1 commit intomainfrom
doc/nh_update-fulcrum-logo

Conversation

@nh13
Copy link
Copy Markdown
Collaborator

@nh13 nh13 commented Apr 9, 2026

Summary

  • Replace old logo with light/dark theme-aware versions using picture element
  • Update badge colors (Email: #38b44a, Visit: #26a8e0)
  • Add "Visit us" text if missing

Summary by CodeRabbit

  • Documentation
    • Enhanced the README with Fulcrum Genomics branding elements in the header, including a linked logo image with automatic light and dark theme variant support
    • Added "Visit us" call-to-action link and badge-style contact links (email and website) to make it easier for users to connect and learn more

@nh13 nh13 requested a review from msto as a code owner April 9, 2026 01:15
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 9, 2026

📝 Walkthrough

Walkthrough

Added branding elements to the README including a Fulcrum Genomics logo with light and dark variants, along with call-to-action links for email contact and website visit. No functional code or documentation content was modified.

Changes

Cohort / File(s) Summary
README Branding Update
README.md
Added Fulcrum Genomics logo (light/dark themed image variants), "Visit us" text link, and two badge-style links for email and website contact in the header area.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A logo hops onto the page,
With badges bright, our brand takes stage,
Dark and light for every sight,
Fulcrum shines both day and night! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The pull request title 'doc: Add Fulcrum Genomics branding to README' accurately and concisely describes the main change: adding branding elements to the README documentation.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch doc/nh_update-fulcrum-logo

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 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`:
- Around line 23-24: The two <img> elements inside the README anchor badges (the
Email_us and Visit_Us badge anchors) are missing alt attributes; update those
<img> tags to include concise, meaningful alt text (e.g., alt="Email us" for the
mailto badge and alt="Visit Fulcrum Genomics" or alt describing the
WordPress/website badge for the Visit_Us badge) so screen readers can convey
their purpose.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ec3889e2-c127-46e7-b12e-2543a0c89203

📥 Commits

Reviewing files that changed from the base of the PR and between 7df73b5 and ecde981.

⛔ Files ignored due to path filters (2)
  • .github/logos/fulcrumgenomics-dark.svg is excluded by !**/*.svg
  • .github/logos/fulcrumgenomics-light.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • README.md

Comment thread README.md
@nh13 nh13 assigned msto Apr 9, 2026
@msto msto assigned nh13 and unassigned msto Apr 9, 2026
@msto msto changed the title doc: add fulcrum genomics branding to README doc: Add Fulcrum Genomics branding to README Apr 9, 2026
@nh13 nh13 merged commit 9c1d358 into main Apr 9, 2026
3 checks passed
@nh13 nh13 deleted the doc/nh_update-fulcrum-logo branch April 9, 2026 16:08
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