Skip to content

New project entry: resume-toolkit#269

Open
zzamboni wants to merge 2 commits intojsonresume:masterfrom
zzamboni:master
Open

New project entry: resume-toolkit#269
zzamboni wants to merge 2 commits intojsonresume:masterfrom
zzamboni:master

Conversation

@zzamboni
Copy link
Copy Markdown

@zzamboni zzamboni commented Mar 23, 2026

https://github.com/zzamboni/resume-toolkit

Pull Request

Description

https://github.com/zzamboni/resume-toolkit:

Easy-to-use toolkit to export JSONResume to both HTML (using a custom version of the Even theme, with additional features) and PDF (using a custom Typst exporter using the brilliant-cv template). Supports generating publications lists (both embedded in the CV and as a standalone page/document) from BibTeX files.

Changes Made

Added project entry to integrations.js

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (code improvement without changing functionality)
  • Documentation update
  • Performance improvement
  • Test coverage improvement
  • Dependency update

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • E2E tests added/updated
  • Manual testing completed

Test Details:

None

Quality Checklist

  • All tests passing locally (pnpm test)
  • No ESLint errors (pnpm lint)
  • Code formatted with Prettier (pnpm prettier)
  • All files ≤150 lines (verify with: find . -name "*.ts" -o -name "*.tsx" -o -name "*.js" | xargs wc -l | awk '$1 > 150')
  • Test coverage maintained or improved
  • No new security vulnerabilities (pnpm audit)
  • Documentation updated (if needed)
  • TypeScript types are proper (no any unless absolutely necessary)

Performance Impact

  • No performance impact
  • Performance improved
  • Potential performance impact (explain below)

Details:

Breaking Changes

  • No breaking changes
  • Breaking changes (describe below)

Details:

Screenshots/Demos

Additional Notes


For Reviewers:

  • Does this PR follow the contribution guidelines?
  • Are all quality checks passing?
  • Is the code well-tested and documented?
  • Are there any security concerns?

Summary by CodeRabbit

  • New Features
    • Added "resume-toolkit" to the projects catalog as a new framework entry.
    • The listing includes a description, link, and language tags (Python, JavaScript, Typst) so users can discover and access this multi-language development tool directly from the projects page.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 23, 2026

⚠️ No Changeset found

Latest commit: 1878a09

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 23, 2026

@zzamboni is attempting to deploy a commit to the JSON Resume Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 709bf42a-dcd4-459b-9a4c-ef1bd5806e85

📥 Commits

Reviewing files that changed from the base of the PR and between ad02127 and 1878a09.

📒 Files selected for processing (1)
  • apps/homepage2/app/projects/data/projects/integrations.js
✅ Files skipped from review due to trivial changes (1)
  • apps/homepage2/app/projects/data/projects/integrations.js

Walkthrough

A new project object for resume-toolkit was appended to the exported projects array, adding description, GitHub link, category: 'framework', and language: 'Python/Javascript/Typst'.

Changes

Cohort / File(s) Summary
Project Data Addition
apps/homepage2/app/projects/data/projects/integrations.js
Appended a new project entry { name: 'resume-toolkit', description, link: 'https://github.com/zzamboni/resume-toolkit', category: 'framework', language: 'Python/Javascript/Typst' } to the exported projects array.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • thomasdavis

Poem

🐰 I hopped through the projects, found a new treat,
Resume-toolkit now sits tidy and neat.
Python, JS, Typst — formats galore,
Added with care, ready to explore! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'New project entry: resume-toolkit' directly describes the main change in the pull request—adding a new project entry for the resume-toolkit project to the integrations.js file.
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

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 and usage tips.

Copy link
Copy Markdown
Contributor

@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 `@apps/homepage2/app/projects/data/projects/integrations.js`:
- Line 178: The long value assigned to the description property in
integrations.js exceeds the 200-character line limit; split the string into
multiple shorter string literals (e.g., broken at logical phrase boundaries) and
concatenate them (or use a multi-line template literal consistent with the file
style) for the description property so each source line is ≤200 chars while
preserving the exact text and punctuation.
🪄 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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: aa183193-e1cd-458c-a08e-64a01040787d

📥 Commits

Reviewing files that changed from the base of the PR and between c8503b5 and ad02127.

📒 Files selected for processing (1)
  • apps/homepage2/app/projects/data/projects/integrations.js

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
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