-
Notifications
You must be signed in to change notification settings - Fork 4
docs(contributing): clarify repository purpose and contribution standards #34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,275 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| # Contributing Guidelines | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Thanks for helping improve the BitDevs Socratic Seminar archive! | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| This document defines the scope, rules, and workflow for contributing to the repository. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Instructions for creating or updating meetup posts live in the README. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| This document explains **what** we accept and **how** to contribute so we stay consistent. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Contents | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Scope of contributions](#scope-of-contributions) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Citation & link policy](#citation--link-policy) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Style guide (Markdown)](#style-guide-markdown) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Contribution workflow](#contribution-workflow) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Branch naming conventions](#branch-naming-conventions) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Commit message conventions](#commit-message-conventions) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Pull Request (PR) process](#pull-request-pr-process) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [Squashing & merge strategy](#squashing--merge-strategy) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Scope of contributions | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| This repository accepts two main types of contributions: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Meetup-related content**: topics, links, summaries, and corrections for specific Socratic seminars. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **General repository maintenance**: documentation, layouts, scripts, CI configuration, and other housekeeping. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| This guide focuses on defining what is in scope and how contributions should be made. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Workflow and naming rules apply to all contribution types. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ### Accepted contributions | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| #### General repository maintenance | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Documentation improvements | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Layout and styling changes | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Scripts, tooling, and CI configuration | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Refactors that improve consistency or maintainability | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| #### Meetup-related contributions | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Topic summaries and background material | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Curated resource lists (papers, PRs, BIPs, mailing-list threads) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Corrections and structural improvements | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - New templates that improve the contributor experience | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ### Out of scope | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Opinionated or editorial content unless explicitly approved | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Content unrelated to Bitcoin or Socratic Seminar discussions | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Promotional or low-signal material | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| If you only want to **propose topics** (without editing files yourself), you can open an Issue like: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| > `Topics for Socratic #NNN` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| and list the suggested links. In that case, the sections **Citation & link policy**, and **Style guide (Markdown)** are the most relevant. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Citation & link policy | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| This archive is meant to point to **public resources** and capture **what was discussed**, not who said what in the room. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| When adding topics: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Prefer **canonical, stable URLs**: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Bitcoin Core / other repos on GitHub | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - BIPs from the official BIPs repository | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - bitcoin-dev / delvingbitcoin threads | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - arXiv papers, project documentation, blog posts | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Avoid: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - URL shorteners whenever possible | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Tracking parameters (`utm_*`, etc.) when it doesn't break the link | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - X/Twitter links are acceptable if they are the main source | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Respect privacy: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Don't link to **private chat logs**, closed groups, or personal DMs | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Don't include personal data about attendees (emails, phone numbers, doxxing, etc.) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - When summarizing ideas from participants, it's fine to keep it anonymous unless they already publish under that identity | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Style guide (Markdown) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| To keep the archive consistent and easy to read: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Headings** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Start sections at `##` and use `###` for subsections. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use a small, predictable set of sections, for example: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `## Avisos` / `## Announcements` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `### Bitcoin`, `### Lightning`, `### Mining`, `### Maneiro` / `### Misc`. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Lists** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use bullet lists for topics, with descriptive link text: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| * [BIP-444: Reduced Data Temporary Softfork](https://github.com/bitcoin/bips/pull/2017) - softfork proposal to reduce block data. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| * [LND 0.20.0-rc2](https://github.com/lightningnetwork/lnd/releases/tag/v0.20.0-beta.rc2) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Prefer **descriptive link text** instead of bare URLs. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Language** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - File content can follow the meetup's usual language (e.g. Portuguese or English). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Commit messages and PR descriptions should always be in English.** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Formatting details** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Try to wrap lines around ~100 characters to keep diffs readable. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Avoid trailing whitespace and unnecessary blank lines. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Keep YAML front-matter and indentation consistent with existing files. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - When adding new files, copy from a recent existing post/template to match structure. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Code & configuration** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - If you touch any scripts or configuration files: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Follow the existing style in that language (indentation, quoting, etc.). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Prefer small, focused changes that are easy to review. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Contribution workflow | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 1. **Open an Issue** (optional but recommended for bigger changes) and describe what you'll add or fix. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 2. **Fork** the repo and create a feature branch: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| git checkout -b docs/2025-10-seminar-notes | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 3. **Make your changes** and commit them with a meaningful message. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 4. **Push your changes** to your forked repository: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| git push origin docs/2025-10-seminar-notes | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 5. **Open a Pull Request (PR)** to the main repository and link it to the Issue (if applicable). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Try to keep each PR focused on **one meetup** or **one logical change** (e.g. "fix broken links in September 2025 post"). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Branch naming conventions | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Use short, lowercase names with words separated by hyphens and, when useful, a simple prefix. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Recommended patterns: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat/...` - new content or features | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `fix/...` - bug fixes or corrections | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `docs/...` - documentation-only changes (README, CONTRIBUTING, templates) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `chore/...` - maintenance, refactors, or build-related changes | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Examples: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat/2025-11-socratic-040` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `docs/add-contributing-guide` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `fix/typo-2025-09-topics` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `chore/reorder-readme` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| If you create the branch from the GitHub UI, try to avoid generic names like `patch-1` and prefer something descriptive. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Commit message conventions | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| We follow a style based on [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/), for example: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat: nov meetup` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat(header): add X (@curitibabitdevs) link to navigation` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat(_posts): arkade announcement` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `fix(_posts): update meetup links` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `fix: remove duplicated topics` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `2025-09 (#68)` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+163
to
+170
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's keep it consistent.
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| General rules: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Write commit messages in **English**. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Start with a **type**, optional **scope**, then a **short description**: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <type>(optional-scope): short description | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Keep the subject line concise (ideally ≤ 50 characters). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use the body (optional) to explain **why** you're changing something, not just **what** changed. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Common types: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat` - new posts, new sections, or new functionality | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `fix` - corrections that change what readers/users see (broken links, wrong dates, incorrect metadata, typos that change meaning) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `docs` - README, CONTRIBUTING, or other documentation-only updates | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `chore` - repository maintenance (moving/renaming files, updating CI or dependencies, non-user-visible cleanups) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `refactor` - improving structure of build/automation scripts without changing their output | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `style` - purely stylistic changes (whitespace, line wrapping, indentation) with no change in meaning | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `test` - adding or adjusting tests (if the repo grows tests later) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
GustavoStingelin marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| In practice: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use `fix` when something is "wrong" from the reader's point of view (404 links, incorrect info, broken formatting in the rendered page). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use `style` when you're only changing formatting/whitespace for readability. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use `refactor` when you're improving the structure of helper scripts or config without changing behavior. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use `chore` for general housekeeping that doesn't fit the above (moving posts, updating CI configs, bumping dependencies). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Examples for this repository: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat(topics): add socratic seminar 040` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `feat(posts): add nov 2025 meetup` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `fix(links): update delvingbitcoin URLs for september` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `docs: add contribution guidelines` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `chore(posts): reorder sections by topic` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| If you are working on a seminar topic file and that's the only change in the PR, a short and focused subject like below is fine: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - `2025-09: add socratic seminar topics` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+182
to
+210
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this all can be removed, the first 15 lines of ## Commit message conventions are clear enough.
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Pull Request (PR) process | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| When opening a PR, please: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 1. **Keep it focused.** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Ideally, one PR = one meetup post or one small set of related fixes. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 2. **Sync with `main`** before opening: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| git fetch upstream | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| git checkout main | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| git pull upstream main | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| git checkout your-branch | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| git rebase main | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+220
to
+226
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We have both |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 3. **Run checks locally** if the repository provides them (e.g. linters, test scripts, or static site build). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Example: `make` / `make test` / `jekyll serve`, as documented in the README. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 4. **Write a clear PR description.** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| A simple template you can follow: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Summary | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ------- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Add topics for the November 2025 Socratic seminar. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Include links to BIPs, PRs, and Delving Bitcoin threads. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Details | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ------- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Meetup date: 2025-11-05 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Sections: Announcements, Bitcoin, Lightning, Misc | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Checklist | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --------- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [ ] Links render correctly in Markdown preview | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - [ ] No private or non-public resources referenced | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 5. **Address review comments.** | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Push follow-up commits to the same branch. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - **Clean history approach**: Feel free to amend commits and force-push with `--force-with-lease` to keep history clean—just leave a comment in the PR describing your changes. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Alternatively, you can add new commits for each review round; maintainers will squash at merge time if needed. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ## Squashing & merge strategy | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| To keep the history readable, we generally: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Use **"Squash and merge"** for most PRs so that each PR becomes a **single logical commit** on `main` (often one commit per meetup or per change-set). | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Occasionally keep multiple commits (regular merge) when the history inside the PR is meaningful on its own. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Force-pushing (`git push --force-with-lease`) to your **own feature branches** is a common practice to keep the commit history clean, especially **before** a PR is opened or when you're squashing WIP commits. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+255
to
+262
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We currently use squash because commit histories are often fragmented or poorly structured. Squashing allows the final commit to mainly reflect the PR title, which the maintainer can adjust before merging. That said, we prefer a clean and well structured commit history, since it allows us to use rebase and merge instead, preserving a clearer and more meaningful history. |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Once a PR already has review comments, prefer adding new commits instead of rewriting history, unless a maintainer explicitly asks you to rebase or clean up the branch. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| What this means for contributors: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - You **don't need** perfectly clean commits while you are iterating on your branch. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - It's nice (but not mandatory) to clean up obvious "WIP" commits before asking for review. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Maintainers will pick an appropriate merge strategy. If we squash, we'll make sure the final commit message follows the conventions above. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Please: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Never push directly to `main`. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - Avoid force-pushing to branches that already have review comments unless you're explicitly asked to rebase/cleanup. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not do this joke on the doc.