doc: clarify meaning of move-only commits #34399

pull pushkarscripts wants to merge 1 commits into bitcoin:master from pushkarscripts:master changing 1 files +4 −0
  1. pushkarscripts commented at 2:09 pm on January 24, 2026: none
    Clarify the definition of “move-only commits” in the coding style guidelines.
  2. doc: clarify meaning of move-only commits
    Clarify what is meant by "move-only commits" in the coding
    style guidelines to make expectations clearer for new contributors.
    eba58ed5f5
  3. DrahtBot added the label Docs on Jan 24, 2026
  4. DrahtBot commented at 2:09 pm on January 24, 2026: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/34399.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  5. l0rinc commented at 2:18 pm on January 24, 2026: contributor
    what’s the problem this solves? You added a potential solution without agreeing on what the problem is.
  6. pushkarscripts commented at 2:34 pm on January 24, 2026: none
    the coding style guidelines reference “move-only commits” without a definition, this change adds one.
  7. maflcko commented at 9:27 am on January 26, 2026: member
    thx, but i think everyone knows what this is. If not, they can ask a search engine or LLM for this trivial question.
  8. maflcko closed this on Jan 26, 2026


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-01-27 06:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me