terrancesli opened a new pull request, #444: URL: https://github.com/apache/rocketmq-dashboard/pull/444
## Summary Two changes bundled into one PR: ### 1. Fix `server/Dockerfile` missing `style/` copy The Dockerfile was missing `COPY style ./style` before `RUN mvn package -DskipTests`. This caused Maven checkstyle to fail during `docker compose build` because it could not find `style/rmq_checkstyle.xml`. **Fix**: Add `COPY style ./style` after `COPY src ./src`. ### 2. Standardize pr-review skill document Rewrote `.claude/skills/pr-review/SKILL.md` with: - **8-stage pipeline checklist** — clear sequential flow from metadata fetch → title check → issue fetch → branch checkout → pre-flight fix → backend build → frontend build → docker deploy - **Stage 5: Dockerfile pre-flight fix** — automatic detection and patching of the missing `style/` copy on PR branches that don't have it - **Title format spec** — standardized `[Studio] type: description` format with regex validation and examples - **Consistent stage naming** — each step is now a named `Stage N` for easier reference -- 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]
