Skip to content

build(nix): pin nixpkgs revision in shell.nix - #1

Open
JPHutchins wants to merge 1 commit into
masterfrom
modernize/01-nix
Open

build(nix): pin nixpkgs revision in shell.nix#1
JPHutchins wants to merge 1 commit into
masterfrom
modernize/01-nix

Conversation

@JPHutchins

@JPHutchins JPHutchins commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Stacked series modernizing the lapsed dhall-python (for eventual upstreaming to s-zeng/dhall-python). Base of the stack.

shell.nix — pin nixpkgs to a fixed nixos-25.05 revision (rev + sha256) instead of the mutable refs/tags/22.11 tarball; pkgconfigpkg-config; drop the darwin.apple_sdk.frameworks.Security stub (a no-op in 25.05).

.github/workflows/ci.yml — replace the unmaintained Dhall generator (builder.dhall, ci.dhall, helpers.dhall, schema.dhall) with a small hand-written workflow: an unfiltered pull_request trigger so CI runs on the whole stacked series (not only PRs targeting master), and one ubuntu job that builds with poetry + maturin and runs pytest.

Kept intentionally small — the workflow tracks the tooling as the stack modernizes it (uv at #2, camas at #7, nextest/matrix at the coverage PR), so every PR stays green.

Warning

LLM Disclosure

This post was authored by claude-opus-4-8[1m] on behalf of @JPHutchins, as the base of a stacked modernization of dhall-python — trimmed to the Nix pin plus the minimal CI needed to give the series green coverage.

@JPHutchins JPHutchins left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

OK

shell.nix:
- pin nixpkgs to a fixed nixos-25.05 revision (rev + sha256) instead of
  the mutable refs/tags/22.11 tarball
- rename the removed `pkgconfig` alias to `pkg-config`; drop the
  `darwin.apple_sdk.frameworks.Security` stub (a no-op in 25.05)

.github/workflows/ci.yml (replacing the unmaintained Dhall generator:
builder.dhall, ci.dhall, helpers.dhall, schema.dhall):
- unfiltered `pull_request` trigger so CI runs on the whole stacked
  series, not only PRs targeting master
- one ubuntu job: build with poetry + maturin and run pytest
- kept intentionally small; the workflow tracks the tooling as the stack
  modernizes it (uv at #2, camas at #7)

Co-Authored-By: claude-opus-4-8[1m] <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019JtM6kqpbjzQB5CVosa5pt
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