Skip to content

nix: Make project config consistent#204

Open
jisantuc wants to merge 7 commits into
DataHaskell:mainfrom
jisantuc:nix/js/update-flake
Open

nix: Make project config consistent#204
jisantuc wants to merge 7 commits into
DataHaskell:mainfrom
jisantuc:nix/js/update-flake

Conversation

@jisantuc

@jisantuc jisantuc commented Jun 6, 2026

Copy link
Copy Markdown
Contributor
  • Make examples depend on granite version consistent with non-examples; non-examples want >= 0.6, examples wanted 0.6.*
  • Remove flake.lock from .gitignore -- this is an important file to have in version control; without it, reproducibility guarantees are out the window (discovered when I got yelled at by nix about the hash being wrong for granite, which seemed like it should have been impossible)
  • Add (almost) the rest of the dataframe-* packages to the package outputs / dev shell

Notes

Testing

  • nix develop in this branch
  • cabal repl <any project except dataframe-fusion>

@jisantuc jisantuc force-pushed the nix/js/update-flake branch 2 times, most recently from b90c222 to a9d69a6 Compare June 7, 2026 00:11
Comment thread dataframe-parquet/src/DataFrame/IO/Parquet.hs Outdated
Comment thread dataframe-learn/src/DataFrame/DecisionTree/Cart.hs
Comment thread flake.lock

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

flake.lock tracks hashes of inputs like nixpkgs to make sure people using nix build with the same package sets. It's an essential file to be in version control.

jisantuc added 5 commits June 30, 2026 17:31
* Make examples depend on granite version consistent with non-examples;
  non-examples want >= 0.6, examples wanted 0.6.*
* Remove `flake.lock` from `.gitignore` -- this is an important file to
have in version control; without it, reproducibility guarantees are out
the window (since, if we're using `nixpkgs-*` as an input, without the
`flake.lock`, the `nixpkgs-*` package set you use won't necessarily be
the same `nixpkgs-*` package set that I use)
* Add (almost) the rest of the dataframe-* packages to the package
outputs / dev shell
@jisantuc jisantuc force-pushed the nix/js/update-flake branch from a9d69a6 to 3104fcc Compare July 1, 2026 01:02
@jisantuc

jisantuc commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

CI failures are driven by missing dataframe-expr-serializer from part of the project config. I haven't really dug in on the CI setup with haskell-ci but I can give fixing that a shot here if that'd be useful, otherwise I think the test instructions in the description are sufficient to indicate that the setup here is consistent

Comment thread flake.nix Outdated
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