Add citation and fix logo#17
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 1 minutes and 17 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughThe PR adds metadata and documentation updates: a DOI field and four new authors with ORCID identifiers to CITATION.cff, and updates the README logo URL while adding status badges for Zenodo DOI, PyPI version, build status, Ruff, and uv. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
README.md (1)
1-1: Use a canonical logo source instead of a fork-hosted raw URL.Line 1 points to
d33bs/iceberg-bioimage; this can break if that fork changes or is removed. Prefer a repo-relative path (if rendering target supports it) orraw.githubusercontent.com/WayScience/iceberg-bioimage/...for stability.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 1, Replace the image src that currently points to "https://raw.githubusercontent.com/d33bs/iceberg-bioimage/main/docs/src/_static/iceberg-bioimage-logo.png" in the README's <img> tag with a stable canonical source: either a repo-relative path (e.g., docs/src/_static/iceberg-bioimage-logo.png) if the rendering target supports relative assets, or the official raw URL under the main org "https://raw.githubusercontent.com/WayScience/iceberg-bioimage/main/docs/src/_static/iceberg-bioimage-logo.png"; keep the same alt text and width attributes to preserve rendering.
🤖 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`:
- Line 6: Wrap the PyPI badge image markdown (the line containing the image URL
https://img.shields.io/pypi/v/iceberg-bioimage) in a link to the package page
(e.g., https://pypi.org/project/iceberg-bioimage/) so the badge becomes
clickable; update the README.md badge line to use the standard linked format
[](https://pypi.org/project/iceberg-bioimage/).
---
Nitpick comments:
In `@README.md`:
- Line 1: Replace the image src that currently points to
"https://raw.githubusercontent.com/d33bs/iceberg-bioimage/main/docs/src/_static/iceberg-bioimage-logo.png"
in the README's <img> tag with a stable canonical source: either a repo-relative
path (e.g., docs/src/_static/iceberg-bioimage-logo.png) if the rendering target
supports relative assets, or the official raw URL under the main org
"https://raw.githubusercontent.com/WayScience/iceberg-bioimage/main/docs/src/_static/iceberg-bioimage-logo.png";
keep the same alt text and width attributes to preserve rendering.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
This PR adds the Zenodo citation and fixes the logo.
Summary by CodeRabbit