feat: Hip blog - #572
Conversation
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
✅ Deploy Preview for hiero-open-source ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds a HIP completion tracking blog post, enables GitHub Flavored Markdown tables in rich text, and updates repository star counts. ChangesHIP blog and rendering
Repository statistics
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ 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 |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Signed-off-by: aceppaluni <aceppaluni@gmail.com>
There was a problem hiding this comment.
@aceppaluni there are a couple typos and . instead of ,
but thanks!
is it preferred to hide the matrix?
|
Either way is fine. Waiting for Michael G to provide link to presentation. Will fix typos. |
Signed-off-by: aceppaluni <aceppaluni@gmail.com>
|
Hey @exploreriii 👋 thanks for the PR! This comment updates automatically as you push changes -- think of it as your PR's live scoreboard! PR Checks✅ DCO Sign-off -- All commits have valid sign-offs. Nice work! ✅ GPG Signature -- All commits have verified GPG signatures. Locked and loaded! ✅ Merge Conflicts -- No merge conflicts detected. Smooth sailing! ✅ Issue Link -- Linked to #571 (assigned to you). 🎉 All checks passed! Your PR is ready for review. Great job! |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@content/posts/tracking-hip-completion.md`:
- Line 32: Replace the malformed HIP Presentation link target in the document,
along with the other occurrences at the referenced locations, with a valid
root-relative asset path or remove the link if no destination is available;
ensure no empty http:// placeholder remains.
- Line 2: Update the post front matter and description wording: use “HIPs”
consistently, write “HIPs (Hiero Improvement Proposals)” with the missing space,
and replace “shed light into” with “shed light on.” Rename the `summary` field
to `abstract` and remove `summary` if present, applying the same corrections to
the corresponding 6-6 post.
In `@src/components/RichText/index.tsx`:
- Around line 71-82: Update the custom th and td renderers in the RichText
component to read style.textAlign and apply the corresponding text-left,
text-center, or text-right class while preserving existing classes. Add
regression coverage verifying left, center, and right GFM column alignments for
both header and data cells.
🪄 Autofix
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 Plus
Run ID: a6634e37-7386-4fba-bddc-b0c012272580
⛔ Files ignored due to path filters (5)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yamlpublic/images/Hiero-Improvement-Proposals.pdfis excluded by!**/*.pdfpublic/images/hip-adoption-funnel.pngis excluded by!**/*.pngpublic/images/hip-completion.pngis excluded by!**/*.pngpublic/images/hip-coverage-matrix.pngis excluded by!**/*.png
📒 Files selected for processing (4)
content/posts/tracking-hip-completion.mdpackage.jsonsrc/components/RichText/index.tsxsrc/data/repository_stats.json
Signed-off-by: Daniel Ntege <danientege785@gmail.com>
Signed-off-by: Ntege Daniel <danientege785@gmail.com>
Signed-off-by: aceppaluni <aceppaluni@gmail.com>
Signed-off-by: aceppaluni <aceppaluni@gmail.com>
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com> Signed-off-by: aceppaluni <aceppaluni@gmail.com> Signed-off-by: Daniel Ntege <danientege785@gmail.com> Signed-off-by: Ntege Daniel <danientege785@gmail.com> Co-authored-by: aceppaluni <aceppaluni@gmail.com> Co-authored-by: Daniel Ntege <danientege785@gmail.com> Signed-off-by: Ntege Daniel <danientege785@gmail.com>
Fixes #571
Adds a blog post on HIP's
Work is completed on my side, but is in draft mode as other authors may want to add to this
@danielmarv @aceppaluni
Summary by CodeRabbit
New Features
Updates