Skip to content

docs: expand CONTRIBUTING guide and add PR template - #106

Open
olalois wants to merge 1 commit into
wraith-protocol:developfrom
olalois:docs/contributing-guide-and-pr-template
Open

docs: expand CONTRIBUTING guide and add PR template#106
olalois wants to merge 1 commit into
wraith-protocol:developfrom
olalois:docs/contributing-guide-and-pr-template

Conversation

@olalois

@olalois olalois commented Jul 27, 2026

Copy link
Copy Markdown

Summary

Expands the bare CONTRIBUTING.md and adds .github/PULL_REQUEST_TEMPLATE.md to address the recurring new-contributor mistakes identified in the issue (wrong base branch, wrong PR target, missing conventional-commit prefix).

Closes #91


Type of change

  • docs — changes to meta-docs (CONTRIBUTING.md, README.md, templates)

Branch checklist

  • I branched from develop, not main
  • This PR targets develop, not main
  • My PR title follows conventional commits format

Content checklist

  • No marketing copy — descriptions explain what, not how great
  • N/A — no MDX pages modified

Snippet checklist

  • N/A — no .mdx files with code fences were added or modified

What's included

CONTRIBUTING.md

  • Branch model section (develop = trunk, main = release) with visual diagram and prominent warning callout
  • Getting started: prerequisites, fork/clone, pnpm install, upstream sync, branch naming convention table
  • Making changes: file structure, writing style rules, MDX conventions
  • Commit messages: conventional commits reference table, examples, and rules
  • Opening a pull request: step-by-step guide + common mistakes table
  • Code and snippet standards: how the checker works, no-check usage policy, Stellar testnet job behaviour
  • CI checks table (blocking vs non-blocking)
  • Review process: squash-merge policy, develop → main release cadence

.github/PULL_REQUEST_TEMPLATE.md

  • Summary field with Closes # pre-filled
  • Type of change checkboxes (6 conventional commit types)
  • Branch checklist targeting the three recurring mistakes from the issue
  • Content checklist (TypeScript / Chain enum / no marketing copy)
  • Snippet checklist with pnpm run check:snippets reminder and no-check justification prompt

- Add comprehensive CONTRIBUTING.md with branch model, conventional
  commits reference, setup instructions, snippet standards, and
  CI check table
- Add .github/PULL_REQUEST_TEMPLATE.md with branch checklist,
  conventional-commit type selector, content checklist, and
  snippet-check reminder

Closes wraith-protocol#91
@drips-wave

drips-wave Bot commented Jul 27, 2026

Copy link
Copy Markdown

@olalois Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

Contributing guide expansion + PR template

1 participant