!jtzdZrQSXtEpeGtzAn:nixos.org

Testing with Nix

88 Members
23 Servers

Load older messages


SenderMessageTime
27 Feb 2024
@qyliss:fairydust.space@qyliss:fairydust.spaceThat makes more sense —-I thought I'd have seen the warning.20:00:26
@qyliss:fairydust.space@qyliss:fairydust.spaceRight20:00:30
@k900:0upti.meK900 But if you do create_machine({"hda": "foo.qcow2"}) or whatever 20:00:37
@k900:0upti.meK900That would get you the warning20:00:44
@qyliss:fairydust.space@qyliss:fairydust.spaceRight20:00:48
@k900:0upti.meK900And now it would blow up20:00:49
@qyliss:fairydust.space@qyliss:fairydust.spaceYeah I agree the non-startCommand keys don't need a deprecation period then.20:01:06
@qyliss:fairydust.space@qyliss:fairydust.spaceThe 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:0upti.meK900I guess that's true20:01:45
@k900:0upti.meK900I can add a hack for that20:01:48
@qyliss:fairydust.space@qyliss:fairydust.spaceAnd then remove it after branch-off.20:01:59
@qyliss:fairydust.space@qyliss:fairydust.spaceUsually we silently add the new API, wait a release, add a warning, wait a release, remove the old one.20:02:17
@qyliss:fairydust.space@qyliss:fairydust.spaceBut 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
@qyliss:fairydust.space@qyliss:fairydust.spaceSo adding a warning straight away should be fine.20:02:49
@k900:0upti.meK900 @Alyssa Ross can you test your stuff against my branch? 20:15:28
@k900:0upti.meK900It should warn now20:15:31
@k900:0upti.meK900And hopefully catch a whole bunch of stupid cases20:15:40
@k900:0upti.meK900(thanks mypy)20:15:42
@qyliss:fairydust.space@qyliss:fairydust.spacecan do20:17:14
@qyliss:fairydust.space@qyliss:fairydust.space(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:0upti.meK900Yeah those should definitely work20:18:40
28 Feb 2024
@qyliss:fairydust.space@qyliss:fairydust.spacek900: as expected, they do.00:21:24
@k900:0upti.meK900Cool06:13:13
@k900:0upti.meK900And does it scream at you06:13:23
@k900:0upti.meK900 @Alyssa Ross 06:13:26
29 Feb 2024
@marijan:matrix.orgmarijan joined the room.07:07:54
1 Mar 2024
@qyliss:fairydust.space@qyliss:fairydust.spacek900: might be good if the warning was yellow bold or something10:12:24
@qyliss:fairydust.space@qyliss:fairydust.spaceit's quite easy to miss10:12:30
@k900:0upti.meK900Do we have anything for that in the test driver already?10:13:05
@qyliss:fairydust.space@qyliss:fairydust.spacewell it prints green messages10:13:34

Show newer messages


Back to Room ListRoom Version: 9