Skip to content

Rename the closure argument closure-inputs to closure-sources in the docs - #90

Closed
clhodapp wants to merge 1 commit into
docs/caisson-nixops4from
docs/closure-sources
Closed

clhodapp wants to merge 1 commit into
docs/caisson-nixops4from
docs/closure-sources

Conversation

@clhodapp

Copy link
Copy Markdown
Contributor

nix-caisson/caisson-core#58 and nix-caisson/caisson#143 rename the closure argument closure-inputs to closure-sources, because the attribute set it carries is the sources a flake passes to mkLib. This pull request moves the documentation and the landing page onto the new name.

The concept page and the deep dive take the same word. The concept page is now titled "Closed-over sources", and its file is concepts/closed-over-sources.md. The deep dive is now titled "How sources are closed over", and its file is deep-dives/how-sources-are-closed-over.md. Every link to either page points at the new path. The reference for mkPackagesOverlay lists sources in the context of a package file, in place of inputs.

The pages keep the word inputs wherever they mean Nix flake inputs, for example the inputs module argument of flake-parts and the follows mechanism.

Running mdbook build on this branch produces no warnings.

This pull request is stacked on #88.

🤖 Generated with Claude Code

A registered file receives the sources its flake passed to mkLib, and
the closure argument that carries them is now named closure-sources.
The concept page and the deep dive take the same word: they are now
"Closed-over sources" (concepts/closed-over-sources.md) and "How
sources are closed over" (deep-dives/how-sources-are-closed-over.md).
The context of a package file built by mkPackagesOverlay carries
`sources` in place of `inputs`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@clhodapp

clhodapp commented Nov 1, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #92.

@clhodapp clhodapp closed this Nov 1, 2026
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.

1 participant