Skip to content

The pyroma hook builds no dependency from source (issue btclib-org/.github#1763) - #1995

Merged
fametrano merged 1 commit into
mainfrom
port-1763-pyroma-no-build
Oct 17, 2026
Merged

fametrano merged 1 commit into
mainfrom
port-1763-pyroma-no-build

Conversation

@fametrano

Copy link
Copy Markdown
Member

The pyroma hook takes --no-build, as section 4 of the organization standard says since btclib-org/.github#1768. A locked package with no wheel for the hook's Python then stops the hook and is named, instead of being built from source over the network.

The comment above the hook credits that to --no-build. After this lands, a commit in btclib-org/.github trims this tree from the BACKLOG row of btclib-org/.github#1763.

Evidence

Check Result
pre-commit run pyroma --all-files and pre-commit run --all-files exit 0 (writer)
suite and sphinx-build -n -W exit 0, under the gate lock (fresh reviewer)
wheels for the pinned Python across the check group's lock closure every compiled package has one (fresh reviewer)
merge-tree with the union driver disabled, against main clean
local review dedicated reviewer CLEARED, then fresh reviewer CLEARED

🤖 Generated with Claude Code

…ithub#1763)

The hook takes --no-build, as section 4 of the organization standard
says. Where a locked package has no wheel for the hook's Python, uv
stops and names it instead of building it from source over the network.
The comment above the hook credits that to --no-build, not to
--only-group.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Signed-off-by: Ferdinando Ametrano <ferdinando@ametrano.net>
@fametrano
fametrano enabled auto-merge (squash) October 16, 2026 20:18

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK cddb2aa

@pmazzocchi pmazzocchi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approves head cddb2aa, following the human's decision to approve every pull request awaiting their review that is otherwise ready.

  • Local review: none run by this session; the approval rests on the bot's ACK and CI.
  • Bot review: ACK cddb2aa553195b0ab7895b68255ab1f41730cdbf, #1995 (review)
  • Every check passed or was skipped on this head.
  • Not checked: the gates were not re-run locally (CI's runs on this head are relied on).

@fametrano
fametrano merged commit c3dae98 into main Oct 17, 2026
21 of 22 checks passed
@fametrano
fametrano deleted the port-1763-pyroma-no-build branch October 17, 2026 13:42
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.

2 participants