Skip to content

RC-442 Move vibebot ruling project from sonar-java - #16

Merged
romainbrenguier merged 1 commit into
masterfrom
romain/rc-442-move-vibebot
Oct 9, 2026
Merged

romainbrenguier merged 1 commit into
masterfrom
romain/rc-442-move-vibebot

Conversation

@romainbrenguier

@romainbrenguier romainbrenguier commented Oct 5, 2026 •

Copy link
Copy Markdown

Summary

  • Move the standalone vibebot Maven project from sonar-java/its/vibebot to ruling_java/vibebot.
  • Preserve the POM, scanner properties, and Java source byte-for-byte; no dependency versions change.
  • Companion sonar-java draft: RC-442 Align ruling project keys and remove vibebot sonar-java#6297. It updates the ruling sources submodule and uses the standard test_project helper.

Validation

  • mvn -B -f its/sources/vibebot/pom.xml -DskipTests package succeeds from the sonar-java workspace.
  • Verified every moved file matches the original exactly.
  • Deep analysis reports no issues in the relocated project files.

@gitar-bot

gitar-bot Bot commented Oct 5, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · Adds the relocated standalone Maven project and scanner configuration to this repository.

Moves the vibebot Maven project from sonar-java/its/vibebot to ruling_java/vibebot with byte-for-byte preservation of the POM, scanner properties, and Java sources. No issues found.

Review coverage

🧪 Functional validation No results

📋 Rules No rules evaluated

Cross-repo coverage 1 repository selected

Cross-repo inspection is incomplete. Unread code may contain additional impacts.

🤖 Auto-approval Not enabled · Set up

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Counting what did not apply, without listing it.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@romainbrenguier
romainbrenguier merged commit ec036ce into master Oct 9, 2026
2 checks passed
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