ci: update some lint infra #36609

pull fanquake wants to merge 6 commits into bitcoin:master from fanquake:ci_updates changing 4 files +7 −5

fanquake commented at 4:28 PM on October 22, 2026: member

Bump uv, ruff, mypy, pyzmq and MLC.
DrahtBot added the label Tests on Oct 22, 2026
DrahtBot commented at 4:29 PM on October 22, 2026: contributor


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



External sites
- Code coverage & benchmarks: https://corecheck.dev/bitcoin/bitcoin/pulls/36609
- Ralph review <img src=https://review.fish.foo/assets/bitcoin/bitcoin/36609.png alt="Ralph review status"> https://git.fish.foo/bitcoin/bitcoin/pulls/36609#author-ralph


Reviews

See the guideline and AI policy for information on the review process.

Type Reviewers

ACK willcl-ark

If your review is incorrectly listed, please copy-paste <code></code> into the comment that the bot should ignore.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #35855 <sub><img src="https://drahtbot.space/ack_count/bitcoin/bitcoin/35855.svg"></sub> (guix: update time-machine by fanquake)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
fanquake requested review from willcl-ark on Oct 26, 2026
fanquake force-pushed on Nov 4, 2026
maflcko commented at 9:44 AM on November 4, 2026: member

Can you mention if there are any relevant changes for us or not? Also could add "lint" in the title?

Otherwise, lgtm if CI is green.
fanquake renamed this:
~~ci: update some infra~~
ci: update some lint infra
on Nov 4, 2026
fanquake commented at 10:04 AM on November 4, 2026: member

any relevant changes for us or not?

Looks like this runs ~35% faster than master, but probably unrelated to any of the changes.

add "lint" in the title?

Added.
ci: use uv 0.13 661b475542
ci: use ruff 0.17 a1053449c6
ci: use mypy 2.4.1 a8520e6433
ci: use MLC 1.3.0 cec51655f3
ci: use pyzmq 27.3.0 80020a9564
lint: set ruff target-version to py310 e9e57d1056
in ci/lint_imagefile:13 in 3fbe135c47 outdated
```
   8 | @@ -9,8 +9,8 @@ FROM mirror.gcr.io/ubuntu:26.04
   9 |  # Pin uv and ruff to minor version to avoid breaking changes
  10 |  # https://docs.astral.sh/uv/reference/policies/versioning/
  11 |  # https://docs.astral.sh/ruff/versioning/
  12 | -COPY --from=ghcr.io/astral-sh/uv:0.11 /uv /uvx /bin/
  13 | -COPY --from=ghcr.io/astral-sh/ruff:0.16 /ruff /bin/
  14 | +COPY --from=ghcr.io/astral-sh/uv:0.13 /uv /uvx /bin/
  15 | +COPY --from=ghcr.io/astral-sh/ruff:0.17 /ruff /bin/
```
hebasto commented at 10:17 PM on November 4, 2026:

From https://github.com/astral-sh/ruff/releases/tag/0.17.0:

Ruff now defaults to Python 3.11 instead of 3.10 when no Python version is configured through target-version or requires-python.

Is this what we want?

willcl-ark commented at 12:00 PM on November 5, 2026:

I think as that says we can simply add target-version = "py310" to ruff.toml as per https://docs.astral.sh/ruff/settings/#target-version to fix on 3.10

fanquake commented at 12:01 PM on November 5, 2026:

I pushed a commit to add something for ruff, but I'm not sure if it matters here, or was left unpinned on purpose? cc @willcl-ark
fanquake force-pushed on Nov 5, 2026
willcl-ark approved
willcl-ark commented at 1:38 PM on November 5, 2026: member

ACK e9e57d1056743454727b8c9b9699b2d3e4aa2aa1

LGTM
fanquake merged this on Nov 5, 2026
fanquake closed this on Nov 5, 2026
fanquake deleted the branch on Nov 5, 2026

Type	Reviewers
ACK	willcl-ark

Contributors

Labels

ci: update some lint infra #36609

External sites

Reviews

Conflicts