Skip to main content

Create Pull Request

Guide for creating well-structured Pull Requests.

Before You Start​

  • Issue is approved and assigned
  • Branch follows naming convention (see git-flow)
  • All commits follow conventional commits
  • Tests pass locally
  • Style is clean locally (see Local style gate below)

Local style gate​

Template-based npm packages and VS Code extensions run CI style-check (eslint / prettier / markdownlint).

Before commit and before push/PR, prefer:

npm run style-fix

That typically runs lint:fix + format + lint:md:fix (exact composition may vary by package scripts).

Do not rely on npm run format alone — markdownlint/eslint failures still burn a CI run.

If style-fix is missing, run the package’s fix scripts that match CI style-check, then re-run npm run style-check locally when available.

PR Title​

Use the same format as commit messages:

<type>(<scope>): <description>

Examples:

feat(parser): add support for nested configs
fix(ui): resolve panel flickering on resize
docs: update installation guide

Target Branch​

Your branchPR targetMerge strategy
feature/*developSquash
bugfix/*developSquash
fork:develop / fork:feature/* / fork:bugfix/*developSquash
release/v*mainMerge commit (created by action)
hotfix/v*mainMerge commit (created by action)
UpmergedevelopMerge commit (created by action)

PR Description​

Each repository has two PR templates — GitHub will prompt you to choose:

  • .github/PULL_REQUEST_TEMPLATE/feature-bugfix.md — for feature and bugfix PRs
  • .github/PULL_REQUEST_TEMPLATE/release.md — for release and hotfix PRs (auto-selected by the action)

Fill out the chosen template completely before requesting review.

Checklist Before Requesting Review​

  • Correct target branch
  • npm run style-fix (or equivalent) run; local style-check clean when available
  • CI checks pass
  • No merge conflicts
  • PR description filled out
  • Linked to issue

Review & Merge​

  • Address all review comments before merge
  • Do not merge your own PR (unless explicitly allowed)
  • Maintainer merges with correct strategy (squash or merge commit per table above)