Skip to content

build(demo): add dependency to commons-demo-processor#42

Merged
paodb merged 2 commits intomasterfrom
demo-sources
Apr 15, 2026
Merged

build(demo): add dependency to commons-demo-processor#42
paodb merged 2 commits intomasterfrom
demo-sources

Conversation

@javier-godoy
Copy link
Copy Markdown
Member

@javier-godoy javier-godoy commented Apr 13, 2026

Summary by CodeRabbit

  • Chores
    • Bumped demo dependency version to 5.3.0 and adjusted the build profile to declare the demo processor as provided scope.
    • No runtime or public API changes; this update affects build configuration only.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 13, 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: cd6155c6-63ed-4c0d-ac56-b4d80ba619e5

📥 Commits

Reviewing files that changed from the base of the PR and between 6d2f99d and 9f9fa1b.

📒 Files selected for processing (1)
  • pom.xml
✅ Files skipped from review due to trivial changes (1)
  • pom.xml

Walkthrough

Bumped flowingcode.commons.demo.version from 5.2.0 to 5.3.0 and added a provided-scoped com.flowingcode.vaadin.addons.demo:commons-demo-processor dependency in the demo-jar Maven profile.

Changes

Cohort / File(s) Summary
Maven Configuration
pom.xml
Updated flowingcode.commons.demo.version property from 5.2.0 to 5.3.0; added com.flowingcode.vaadin.addons.demo:commons-demo-processor dependency with provided scope in the demo-jar profile.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 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 accurately describes the main change: adding a dependency to commons-demo-processor in the demo Maven profile.
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
  • Commit unit tests in branch demo-sources

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.

@javier-godoy javier-godoy marked this pull request as ready for review April 13, 2026 17:49
@javier-godoy javier-godoy requested a review from scardanzan April 13, 2026 17:55
@sonarqubecloud
Copy link
Copy Markdown

@paodb paodb merged commit b470c12 into master Apr 15, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from To Do to Pending release in Flowing Code Addons Apr 15, 2026
@paodb paodb deleted the demo-sources branch April 15, 2026 15:09
@paodb paodb moved this from Pending release to Done in Flowing Code Addons Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants