27 Feb 2024 |
Alyssa Ross | Right | 20:00:48 |
K900 ⚡️ | And now it would blow up | 20:00:49 |
Alyssa Ross | Yeah I agree the non-startCommand keys don't need a deprecation period then. | 20:01:06 |
Alyssa Ross | The reason to do a deprecation period is usually to make it possible to have some code that works with both stable and unstable. | 20:01:34 |
K900 ⚡️ | I guess that's true | 20:01:45 |
K900 ⚡️ | I can add a hack for that | 20:01:48 |
Alyssa Ross | And then remove it after branch-off. | 20:01:59 |
Alyssa Ross | Usually we silently add the new API, wait a release, add a warning, wait a release, remove the old one. | 20:02:17 |
Alyssa Ross | But I don't think that's necessary here because warnings from the test driver are not nearly as disruptive as warnings from Nix. | 20:02:31 |
Alyssa Ross | So adding a warning straight away should be fine. | 20:02:49 |
K900 ⚡️ | @Alyssa Ross can you test your stuff against my branch? | 20:15:28 |
K900 ⚡️ | It should warn now | 20:15:31 |
K900 ⚡️ | And hopefully catch a whole bunch of stupid cases | 20:15:40 |
K900 ⚡️ | (thanks mypy) | 20:15:42 |
Alyssa Ross | can do | 20:17:14 |
Alyssa Ross | (My stuff is https://spectrum-os.org/git/spectrum/tree/release/checks/try.nix?id=0b84f07c366366e26fa4047bac1c47037450d501 and https://spectrum-os.org/git/spectrum/tree/release/checks/no-roothash.nix?id=0b84f07c366366e26fa4047bac1c47037450d501 btw) | 20:17:51 |
K900 ⚡️ | Yeah those should definitely work | 20:18:40 |
28 Feb 2024 |
Alyssa Ross | k900: as expected, they do. | 00:21:24 |
K900 ⚡️ | Cool | 06:13:13 |
K900 ⚡️ | And does it scream at you | 06:13:23 |
K900 ⚡️ | @Alyssa Ross | 06:13:26 |
29 Feb 2024 |
| marijan joined the room. | 07:07:54 |
1 Mar 2024 |
Alyssa Ross | k900: might be good if the warning was yellow bold or something | 10:12:24 |
Alyssa Ross | it's quite easy to miss | 10:12:30 |
K900 ⚡️ | Do we have anything for that in the test driver already? | 10:13:05 |
Alyssa Ross | well it prints green messages | 10:13:34 |
Alyssa Ross | but you could just embed the escape sequences in the message | 10:13:45 |
K900 ⚡️ | Oh OK it has colorama | 10:14:33 |
K900 ⚡️ | @Alyssa Ross better? https://github.com/NixOS/nixpkgs/pull/292514 | 10:21:03 |
| tomberek set a profile picture. | 15:21:09 |