wu-sheng opened a new pull request, #51: URL: https://github.com/apache/skywalking-horizon-ui/pull/51
## Why Opening a PR here has a fixed preflight checklist (CHANGELOG + docs in the one-line house style, type-check + build, new UI strings translated across all eight locales, license headers, lint + unit tests, live-OAP validation) and a fixed post-merge cleanup. Capture it as a reusable `/pull-request` skill so it runs the same way every time, alongside the existing `local-boot` / `build-dashboard` skills. ## What Adds `.claude/skills/pull-request/SKILL.md`: - **Preflight gate** mirroring `.github/workflows/ci.yaml` — `pnpm -r run type-check`, `ui`/`bff` builds, `pnpm -r run test:unit`, `pnpm license:check`, plus the i18n step (author `en.json` first, then fill `zh-CN/es/pt/ja/ko/de/fr`, validate template overlays with `i18n:validate`) and a key-diff snippet to catch untranslated UI keys. - **Commit/PR rules** — branch off `main`, push directly to `origin` (committer access), and never add AI co-author / "Generated with Claude Code" trailers (per `CLAUDE.md`). - **Post-merge cleanup** — confirm `MERGED`, `git checkout main`, `git pull --ff-only`, delete the merged local branch (`-d`, falling back to `-D` for squash-merges), `git remote prune origin`. ## Validation Docs-only skill file. `pnpm license:check` clean (Markdown is excluded from license headers); no code, i18n, or wire surface touched. Dogfooded: this PR itself was opened via the skill's workflow, and PR #50's post-merge cleanup followed its step 4. -- 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]
