Skip to content

platformatic-kafka 1.29#413

Merged
kibertoad merged 1 commit intomainfrom
chore/plt_kafka_update
Mar 4, 2026
Merged

platformatic-kafka 1.29#413
kibertoad merged 1 commit intomainfrom
chore/plt_kafka_update

Conversation

@CarlosGamero
Copy link
Collaborator

@CarlosGamero CarlosGamero commented Mar 4, 2026

Summary by CodeRabbit

  • Chores
    • Updated @platformatic/kafka dependency to version 1.29.0
    • Removed @platformatic/dynamic-buffer dependency

@CarlosGamero CarlosGamero self-assigned this Mar 4, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

📝 Walkthrough

Walkthrough

The pull request updates the @platformatic/kafka dependency from version ^1.28.0 to ^1.29.0 and removes the @platformatic/dynamic-buffer dependency from the packages/kafka/package.json file.

Changes

Cohort / File(s) Summary
Dependency Updates
packages/kafka/package.json
Bumped @platformatic/kafka from ^1.28.0 to ^1.29.0 and removed @platformatic/dynamic-buffer dependency.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • kibertoad

Poem

🐰 A hop, skip, and version bump so fine,
Dependencies dance in perfect line,
Old buffers fade, new kafka flows,
A rabbit's code in orderly rows! ✨

🚥 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 'platformatic-kafka 1.29' directly references the main change: updating @platformatic/kafka to version 1.29.0.
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
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/plt_kafka_update

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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

@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 `@packages/kafka/package.json`:
- Around line 53-57: The dependency entry for "@platformatic/kafka" in the
dependencies block is pointing to a non-existent version (^1.29.0); update the
version to an existent release (e.g., change "@platformatic/kafka" to "^1.28.0")
or, if you intentionally need a pre-release, replace the version with the
correct pre-release tag (or add the appropriate registry/dist-tag) and confirm
that the package will be resolvable by npm; ensure the change is made in the
dependencies object in package.json where "@platformatic/kafka" is declared.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5d49faef-ee5e-4157-9daf-e32adca65e61

📥 Commits

Reviewing files that changed from the base of the PR and between 5d2c339 and 13a2561.

📒 Files selected for processing (1)
  • packages/kafka/package.json

@kibertoad kibertoad merged commit 0a427a6 into main Mar 4, 2026
10 of 12 checks passed
@kibertoad kibertoad deleted the chore/plt_kafka_update branch March 4, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants