Skip to content

Remove WorkerProto::Op::HasSubstitutes#14988

Merged
Ericson2314 merged 1 commit intomasterfrom
remove-has-substitutes
Jan 14, 2026
Merged

Remove WorkerProto::Op::HasSubstitutes#14988
Ericson2314 merged 1 commit intomasterfrom
remove-has-substitutes

Conversation

@edolstra
Copy link
Member

Motivation

This operation has been deprecated since 09a6321 (July 2012). It was used by client versions <= 11, which is below MINIMUM_PROTOCOL_VERSION (currently 18).

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

This operation has been deprecated since
09a6321 (July 2012). It was used by
client versions <= 11, which is below `MINIMUM_PROTOCOL_VERSION`
(currently 18).
@edolstra edolstra requested a review from Ericson2314 as a code owner January 13, 2026 11:30
Copy link
Member

@Ericson2314 Ericson2314 left a comment

Choose a reason for hiding this comment

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

Not use in Hydra either

@Ericson2314 Ericson2314 added this pull request to the merge queue Jan 14, 2026
Merged via the queue into master with commit 609a4e9 Jan 14, 2026
18 checks passed
@Ericson2314 Ericson2314 deleted the remove-has-substitutes branch January 14, 2026 17:59
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