Skip to content

doc: add fulcrum genomics branding to README#63

Merged
nh13 merged 2 commits intomainfrom
doc/nh_update-fulcrum-logo
Apr 9, 2026
Merged

doc: add fulcrum genomics branding to README#63
nh13 merged 2 commits 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 README with promotional content and visual elements
  • Chores

    • Updated GitHub Actions workflow configurations to pin external action versions for improved stability and security

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 9, 2026

📝 Walkthrough

Walkthrough

This pull request pins GitHub Actions versions to specific commit SHAs across four workflow files for reproducible and secure builds, and adds Fulcrum Genomics promotional content to the README.

Changes

Cohort / File(s) Summary
GitHub Actions Version Pinning
.github/workflows/integration-tests.yml, .github/workflows/publish-logger-plugin.yml, .github/workflows/publish.yml, .github/workflows/tests.yml
Replaced floating GitHub Actions version tags with pinned commit SHAs (e.g., actions/checkout@v4 → specific SHA) across all workflow files to enhance reproducibility and supply chain security. No step logic or inputs modified.
README Promotional Update
README.md
Added Fulcrum Genomics promotional section with branded logo (light/dark variants), website link, and call-to-action badges inserted after the introductory paragraph.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 Commits pinned, workflows tight,
Security strengthens day and night,
README blooms with badges bright,
Fulcrum's logo shines its light!
A rabbit's work in progress done,
Reproducible builds for everyone! 🔐✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description provides a summary of changes but does not follow the provided template structure, missing required sections like Type of Change, Checklist, Testing, Related Issues, and Additional Notes. Restructure the description to match the template: include a proper 'Type of Change' section (Documentation update), complete the Checklist section, add Testing, Related Issues, and Additional Notes sections.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The PR title 'doc: add fulcrum genomics branding to README' accurately reflects the primary change documented in the raw summary: promotional content was added to the README.
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 doc/nh_update-fulcrum-logo

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

Required by fg-labs org policy: all actions must be pinned to a
full-length commit SHA rather than a tag.
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 32-33: The two raw HTML badge <img> elements are missing alt
attributes (tripping MD045); update both <img> tags inside the anchor elements
so they include meaningful alt text (e.g., alt="Email us badge" and alt="Visit
us badge" or more descriptive text) to improve accessibility and satisfy the
lint rule; ensure you add the alt attribute to the same <img> tags currently
used for the Email and Visit badges.
🪄 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: 2307509e-09fd-4949-848a-54fa12a383fb

📥 Commits

Reviewing files that changed from the base of the PR and between 453d660 and 495020a.

⛔ 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 (5)
  • .github/workflows/integration-tests.yml
  • .github/workflows/publish-logger-plugin.yml
  • .github/workflows/publish.yml
  • .github/workflows/tests.yml
  • README.md

@nh13 nh13 merged commit cb94991 into main Apr 9, 2026
6 checks passed
@nh13 nh13 deleted the doc/nh_update-fulcrum-logo branch April 9, 2026 07:09
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.

1 participant