Silvan Mosberger

Results 111 comments of Silvan Mosberger

This is a bit messy, but I wrote an inline script to check whether the PR is mergeable, and it seems to work decently: https://github.com/NixOS/nixpkgs/blob/c8db8bd9656ee3d373ce9445063c25c47f442118/.github/workflows/check-by-name.yml#L31-L86

Some features that should be supported: - Phases can produce an output at execution time - The Nix evaluation of phases can depend on the outputs of previous phases -...

Gave this another implementation try in https://github.com/Infinisil/nix/commit/5dbf67cab037b3bb8302052fdda6534e5cf23d9b, which works much better than the previous one. For example: ```nix let lib = import ./lib; set1 = lib.makeExtensible (self: builtins.trace "set1" {...

Turns out there was no infinite recursion in the last version, but instead it was so slow that I didn't think it would ever finish! I worked more on this,...

Opened a draft PR to track further development: https://github.com/NixOS/nix/pull/4154

Problem: Lorri can't be used with https://github.com/nix-community/yarn2nix, because a nix-shell for that requires a `shellHook` like ``` ln -s ${yarnPkg}/libexec/myModule/node_modules node_modules ``` such that `node_modules` in the current directory contains...

With NPM 7, hook scripts, which this PR relies on, are not supported anymore, see also https://github.com/nix-community/npmlock2nix/issues/110. However with #151 it will become possible to patch the sources of arbitrary...

Needs to be run in CI yet though: https://github.com/NixOS/nixfmt/issues/165

While it has been configurable for a while using `-w`, since #118, it's a _soft_ limit, and not a hard one. We should also implement a way to configure the...

Ah I see, makes sense! I guess for that we should use [editorconfig's `indent_size`](https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties#indent_size)