purescript / registry-dev

Development work related to the PureScript Registry
https://github.com/purescript/registry
97 stars 80 forks source link

Run `nix flake update` #572

Closed JordanMartinez closed 1 year ago

JordanMartinez commented 1 year ago

See https://github.com/purescript/registry/issues/121#issuecomment-1352404601


$ nix flake update --extra-experimental-features nix-command --extra-experimental-features flakes
warning: updating lock file './registry-dev/flake.lock':
• Updated input 'flake-compat':
    'github:edolstra/flake-compat/b4a34015c698c7793d592d66adbab377907a2be8' (2022-04-19)
  → 'github:edolstra/flake-compat/009399224d5e398d03b22badca40a37ac85412a1' (2022-11-17)
• Updated input 'nixpkgs':
    'github:nixos/nixpkgs/5d3976195b04db1d33adf7528f7a3c8a66b9f818' (2022-11-03)
  → 'github:nixos/nixpkgs/9264e62367d4893af624b2cbd163e843e7271bf1' (2022-12-14)
warning: Git tree '/home/jordan/programming/registry-dev' is dirty
thomashoneyman commented 1 year ago

Shoot, looks like we're using @f-f's fork and it doesn't support 0.15.7: https://github.com/purescript/registry-dev/blob/412da4527ff833b9de08b26e4cc670fb4c4b73a0/flake.nix#L14-L18

We can either update that fork, or we can add both the fork and justinwoo's easy-purescript-nix to our Flake, taking all purescript tools except spago from easy-purescript-nix and taking only spago from the f-f fork.

f-f commented 1 year ago

I am pushing a new Spago version later today - it will be the first alpha so I can push it to my fork or add it to easy-purescript-nix

f-f commented 1 year ago

Update: I am still going to push the first Spago alpha later, but I synced my fork to upstream in the meantime.

JordanMartinez commented 1 year ago

Update: I am still going to push the first Spago alpha later, but I synced my fork to upstream in the meantime.

Does that mean I don't need to do this? Or that I should run the above command again?

f-f commented 1 year ago

Yep, now my fork is up to date (both synced with upstream, and with the latest of the new spago).

Running the command again should have worked, but you were unlucky to catch the brief window in which I pushed a broken version of spago. Running again should fetch the latest version that has the fix.

JordanMartinez commented 1 year ago

CI passes. Can we get this merged?