chennaji9 opened a new pull request, #5208:
URL: https://github.com/apache/rocketmq-dashboard/pull/5208
### Which Issue(s) This PR Fixes
None — documentation accuracy fix found by inspection.
### Brief Description
README, README_zh and CONTRIBUTING all claim that `mvn test` runs ArchUnit
hexagonal-architecture checks ("ArchUnit enforced", "a violation fails the
build"). `server/pom.xml` declares `archunit-junit5`, but no `ArchRule`,
`@ArchTest`, `@AnalyzeClasses` or `ClassFileImporter` exists anywhere under
`server/src/test`, so no architecture check ever runs today. The docs now state
the current reality: the layering is enforced through review, and the
dependency is declared but not wired up. Wording is kept consistent between the
English and Chinese READMEs and CONTRIBUTING.
(`.github/PULL_REQUEST_TEMPLATE.md` also carries a checklist line that
references the ArchUnit checks; it is left untouched here to keep this PR
docs-only — happy to follow up if wanted.)
Base is `rocketmq-studio` per CONTRIBUTING.md (the PR template's base
comment is outdated).
### How Did You Test This Change?
- `grep -ri archunit server/src` — no matches in any Java source; the only
reference is the dependency declaration in `server/pom.xml`.
- `grep -rn "ArchRule\|@ArchTest\|AnalyzeClasses\|ClassFileImporter"
server/src/test` — no matches.
- Documentation-only change; no build or tests affected. Markdown reviewed
in both languages for consistency.
### Checklist
- [x] One coherent change; unrelated modifications are not bundled in
- [x] Commit subject follows Conventional Commits (`docs:`)
- [ ] Tests added or updated for non-trivial changes (n/a — docs only)
- [x] New UI text has both Chinese and English entries under `web/src/i18n/`
(n/a — no UI change)
- [x] Architecture constraints stay green (n/a — no Java change)
- [x] New source files carry the ASF license header (n/a — no new source
file)
- [x] Documentation touched where behaviour changed (this PR is the doc fix)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
