Skip to content

fix(deps): align feign micrometer with spring cloud#302

Open
Komzpa wants to merge 1 commit intomasterfrom
codex/fix-noclassdeffounderror-in-prod-event-api
Open

fix(deps): align feign micrometer with spring cloud#302
Komzpa wants to merge 1 commit intomasterfrom
codex/fix-noclassdeffounderror-in-prod-event-api

Conversation

@Komzpa
Copy link
Copy Markdown
Member

@Komzpa Komzpa commented Sep 8, 2025

Summary

  • pin feign-micrometer to 11.10 to match Spring Cloud's Feign version and prevent NoClassDefFoundError

Testing

  • make precommit

https://chatgpt.com/codex/tasks/task_e_68bd6ae391c8832491e7d3bdf95c1506

Summary by CodeRabbit

  • Chores
    • Updated a third-party library version used for metrics/monitoring integration. No functional changes are expected for end-users, and existing behavior should remain consistent.
    • This change affects build-time dependencies only and does not introduce new features or UI updates.
    • If you notice any unexpected behavior after the update, please report it so we can investigate.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Sep 8, 2025

Walkthrough

The Maven property feign-micrometer.version in pom.xml was updated from 13.6 to 11.10, altering the resolved version of the feign-micrometer dependency. No other build configuration changes are indicated.

Changes

Cohort / File(s) Summary of Changes
Build configuration
pom.xml
Updated property feign-micrometer.version from 13.6 to 11.10, affecting the resolved feign-micrometer dependency version.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 01df8dd and ebd932e.

📒 Files selected for processing (1)
  • pom.xml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: build
  • GitHub Check: build
🔇 Additional comments (1)
pom.xml (1)

24-24: Approve feign-micrometer.version pin; manual verification required

  • Pinning to 11.10 matches Spring Cloud OpenFeign 3.1.9’s feign-core 11.10.
  • Maven isn’t available in this environment—please confirm all io.github.openfeign dependencies resolve to 11.10.
  • Optional: import the Feign BOM at 11.10 in dependencyManagement to prevent future drift.
  • Spring Cloud 2021.0.x is EOL; plan an upgrade to 2023.x/2024.x (removing this pin).
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/fix-noclassdeffounderror-in-prod-event-api

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.

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.

1 participant