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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending release

Development

Successfully merging this pull request may close these issues.

2 participants