| 21 Nov 2023 |
infinisil | * Updated https://github.com/NixOS/nix.dev/pull/802 and marked as ready for review, looking pretty good now imo :) | 02:40:52 |
| brainrake left the room. | 15:27:59 |
| Eelco changed their display name from niksnut to Eelco. | 16:37:09 |
infinisil | fricklerhandwerk: Unless the github deployments thing is necessary for netlify (I don't think it is?), please turn it off (which needs repo admin permissions afaik), it's spamming events for every PR update: https://github.com/NixOS/nix.dev/pull/802 | 18:26:02 |
asymmetric | In reply to @infinisil:matrix.org fricklerhandwerk: Unless the github deployments thing is necessary for netlify (I don't think it is?), please turn it off (which needs repo admin permissions afaik), it's spamming events for every PR update: https://github.com/NixOS/nix.dev/pull/802 i think it comes from here: https://github.com/NixOS/nix.dev/blob/master/.github/workflows/build-and-deploy.yml | 18:38:55 |
asymmetric | the action has an enable-github-deployment option which defaults to true. we could make it dependent on whether the push is to master instead | 18:39:46 |
asymmetric | * the netlify action has an enable-github-deployment option which defaults to true. we could make it dependent on whether the push is to master instead | 18:40:01 |
infinisil | asymmetric: Ah I see, but what's the point of these github deployments anyways? The comment it posts in the PR always contains an up-to-date link to the netlify deployment | 18:41:22 |
infinisil | * asymmetric: Ah I see, but what's the point of these github deployments anyways? The comment it posts in the PR always contains an up-to-date link to the netlify deployment: https://github.com/NixOS/nix.dev/pull/802#issuecomment-1817733137 | 18:41:35 |
asymmetric | don't think there's any special utility tbh.. other than getting a view of all deployments, and a summary in the project's sidebar | 18:43:59 |
infinisil | asymmetric: https://github.com/NixOS/nix.dev/pull/803 | 18:54:03 |
asymmetric | Do you get a This branch has not been deployed spammy message too? | 18:54:55 |
infinisil | asymmetric: Oh yeah, interesting | 18:55:22 |
infinisil | So maybe it doesn't work at all if that's disabled.. | 18:55:38 |
infinisil | I just want it to post the comment | 18:56:02 |
infinisil | Oh maybe wait until the action is done | 18:56:17 |
asymmetric | it did deploy to netlify, it's just that github doesn't know about it | 18:56:27 |
infinisil | Yup there it is: https://github.com/NixOS/nix.dev/pull/803#issuecomment-1821480699 :D | 18:56:28 |
asymmetric | we would probably have to disable github deployments altogether (needs an admin) to get rid of that message | 18:56:59 |
infinisil | The message is still there though 🤔 | 18:57:02 |
asymmetric | In reply to @asymmetric:matrix.dapp.org.uk it did deploy to netlify, it's just that github doesn't know about it yeah it's because of this | 18:57:18 |
infinisil | Yeah, let's see if there's a setting for this in a personal repo | 18:57:23 |
| * infinisil can't find any setting | 19:02:11 |
infinisil | But it's probably related to "Environments" | 19:02:41 |
@bzzm3r:matrix.org | Redacted or Malformed Event | 19:09:40 |
@bzzm3r:matrix.org | (sorry, wrong channel!) | 19:13:28 |
Artturin | https://github.com/NixOS/nixpkgs/pull/265478 | 22:20:54 |
| 22 Nov 2023 |
| @zarel_it:matrix.org left the room. | 09:40:37 |
@olafklingt:matrix.org | I am trying to improve the reproducible nixos-generate-config. it would be nice if we could not reproduce the installer configuration but by use the installer configuration as defined in
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/installer/cd-dvd/installation-cd-graphical-base.nix ... but i have no idea how to do that. (i already commented on this on the ticket but i think here is a better place if someone has an idea) | 15:46:22 |
@olafklingt:matrix.org | * I am trying to improve the reproducible nixos-generate-config. it would be nice if we could not reproduce the installer configuration but use the installer configuration as defined in
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/installer/cd-dvd/installation-cd-graphical-base.nix ... but i have no idea how to do that. (i already commented on this on the ticket but i think here is a better place if someone has an idea) | 15:57:44 |