Skip to content

Add jevtraces - an alpha OTel trace processor - #78

Closed
ishantanu wants to merge 1 commit into
hellogumbo:mainfrom
ishantanu:add-jevtraces
Closed

ishantanu wants to merge 1 commit into
hellogumbo:mainfrom
ishantanu:add-jevtraces

Conversation

@ishantanu

@ishantanu ishantanu commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Project:
https://github.com/ishantanu/jevtraces

Where Jev fits:

  • I added one object to data/projects.json and ran npm run validate
  • I did not commit README.md or site/index.html (CI regenerates them)
  • I am the author, or I linked where the project was announced

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • New Features
    • Added the JevTraces project to the project directory.
    • Listed its OpenTelemetry integration, Go implementation, and span annotation capabilities.

Signed-off-by: Shantanu Deshpande <shantanud106@gmail.com>
@coderabbitai

coderabbitai Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c4fdc1db-ebfa-459e-b4e7-58cbde60b754

📥 Commits

Reviewing files that changed from the base of the PR and between 931021f and 80fa09e.

📒 Files selected for processing (1)
  • data/projects.json

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The project catalog adds jevtraces, an OpenTelemetry Collector processor integration backed by Jev.

Changes

Project catalog

Layer / File(s) Summary
Add jevtraces catalog entry
data/projects.json
Adds jevtraces with its repository, integration category, Go language, star count, added date, and processor description.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: scald

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes the project URL and required checklist confirmations, but it does not complete the required “Where Jev fits” one-sentence explanation. Add one sentence explaining how Jev fits in jevtraces, then retain the completed checklist and project URL.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies the addition of the alpha jevtraces OpenTelemetry trace processor.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

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.

scald pushed a commit that referenced this pull request Sep 24, 2026
Landed from #78.
@scald

scald commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Landed on main as 1d78d38 with you as the commit author, now live on https://awesomejev.com. Closing rather than merging because every PR appends to the same data file and they conflict with each other; landing directly keeps your authorship. Thanks!

@scald scald closed this Sep 24, 2026
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