CI: Allow Version field in checks as per BIP 3 #1891

pull nothingmuch wants to merge 1 commits into bitcoin:master from nothingmuch:allow-version-field changing 1 files +5 −0
  1. nothingmuch commented at 12:58 pm on July 8, 2025: contributor
    BIP 3 introduced a Version field in the header with semver inspired syntax. This PR allows the inclusion of this field and checks that it matches the <MAJOR.MINOR.PATCH> format.
  2. Allow `Version` field in checks as per BIP 3 4f46c5fb3b
  3. murchandamus commented at 2:57 pm on July 8, 2025: contributor

    Thanks, this looks great!

    There is just the hitch that BIP 3 is “Proposed”, but not active yet. I’m working on a PR to suggest activation of BIP 3 in #1820. Seeing your PR, I had actually overlooked that I wasn’t adding the Version field to the buildtable script, so thanks for putting this together. I would propose to cherry-pick this commit into #1820, if that’s okay for you, because the Version field doesn’t exist in the BIP 2 process, and we will need to allow it when BIP 3 becomes active. Alternatively, we could merge your PR alongside #1820, when BIP 3 is activated.

    Meanwhile it seems fine to me for BIP authors to use the Changelog section without putting the Version field in the preamble, and we can then backfill the version fields to the corresponding BIPs after BIP 3 activates.

  4. murchandamus added the label Process on Jul 8, 2025
  5. nothingmuch commented at 3:18 pm on July 8, 2025: contributor

    I would propose to cherry-pick this commit into #1820, if that’s okay for you, because the Version field doesn’t exist in the BIP 2 process, and we will need to allow it when BIP 3 becomes active. Alternatively, we could merge your PR alongside #1820, when BIP 3 is activated.

    Sure, I would prefer whatever minimizes hassle for BIP editors, so sounds like cherry-picking into 1820 and removing the version field from 1890 for now makes the most sense, but I defer to your judgement

  6. murchandamus commented at 8:11 pm on July 8, 2025: contributor
    Thanks, I added it to the PR #1820.
  7. murchandamus closed this on Jul 8, 2025


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-07-11 13:10 UTC

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