Skip to content

Remove path setting#15357

Merged
Ericson2314 merged 2 commits intoNixOS:masterfrom
obsidiansystems:remove-path-setting
Feb 27, 2026
Merged

Remove path setting#15357
Ericson2314 merged 2 commits intoNixOS:masterfrom
obsidiansystems:remove-path-setting

Conversation

@amaanq
Copy link
Member

@amaanq amaanq commented Feb 26, 2026

Motivation

This PR removes the unused OptionalPathSetting and replaces PathSetting with StoreDirSetting.

Context


Add 👍 to pull requests you find important.

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

@github-actions github-actions bot added the store Issues and pull requests concerning the Nix store label Feb 26, 2026
@amaanq amaanq force-pushed the remove-path-setting branch from 2f5ab99 to d67d5b9 Compare February 26, 2026 22:07
@amaanq amaanq force-pushed the remove-path-setting branch from d67d5b9 to 947c413 Compare February 26, 2026 22:15
@amaanq amaanq force-pushed the remove-path-setting branch from 947c413 to 944e945 Compare February 26, 2026 22:19
auto-merge was automatically disabled February 26, 2026 22:34

Head branch was pushed to by a user without write access

@amaanq amaanq force-pushed the remove-path-setting branch from 944e945 to c165ae9 Compare February 26, 2026 22:34
@Ericson2314 Ericson2314 added this pull request to the merge queue Feb 26, 2026
Merged via the queue into NixOS:master with commit c522f58 Feb 27, 2026
15 checks passed
@Ericson2314 Ericson2314 deleted the remove-path-setting branch February 27, 2026 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

store Issues and pull requests concerning the Nix store

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants