23 Sep 2024 |
Pol | https://github.com/NixOS/nixpkgs/pull/343900 | 07:21:21 |
Pol | patka: Yeah :) | 07:21:30 |
Pol | patka: I've also fixed this: https://github.com/NixOS/nixpkgs/pull/343900/commits/3bcaabbb7119cb06f7f79770baf2e11ad293955c | 07:21:45 |
@patka:envs.net | Oopsie, thanks! | 07:23:59 |
@patka:envs.net | Just have a small question on the PR above. Apart from that it looks good. Genghiz would you be able to test that it works with the firefly-iii work? | 07:25:48 |
@patka:envs.net | * Just have a small question on the PR above. Apart from that it looks good. Genghiz would you be able to test/validate that it works with the firefly-iii work? | 07:26:05 |
Genghiz | In reply to @patka:envs.net Just have a small question on the PR above. Apart from that it looks good. Genghiz would you be able to test/validate that it works with the firefly-iii work? Gimme an hour I’ll do that. | 07:29:47 |
Pol | Quick question, how do you usually test PR ? Do you cherry pick the commit in your branch or do another trick ? | 07:32:17 |
Genghiz | In reply to @drupol:matrix.org Quick question, how do you usually test PR ? Do you cherry pick the commit in your branch or do another trick ? I use nixpkgs-review | 09:37:35 |
Pol | But in this case, you need to test this against fireflii, how do you do then? | 09:37:58 |
Genghiz | Nixpkgs-review pr -p firefly-iii
| 09:38:22 |
Pol | nice | 09:39:29 |
Pol | I just notice a small typo | 09:39:35 |
Pol | I will fix it after that PR is merged. | 09:39:42 |
Pol | setComposeRootVersion -> setComposerRootVersion | 09:40:01 |
Pol | setComposeEnvVariables -> setComposerEnvVariables | 09:40:12 |
Pol | Let me do it within this PR | 09:40:23 |
Genghiz | So firefly-iii seems to be fixed. | 09:41:06 |
Pol | Done, I just fixed the typo | 09:41:28 |
Genghiz | Once you merge I’ll close the firefly-iii fix PR. | 09:41:29 |
Pol | Can you try it again? | 09:41:40 |
Pol | And post some feedback in the PR ? | 09:41:45 |
Genghiz | Same PR number? | 09:41:48 |
Pol | yeap | 09:41:54 |
Genghiz | 2 mins | 09:42:10 |
Genghiz | Fwiw the tests work, but I’ll add a properly formatted review message in a bit so just merge post that I think. | 10:21:19 |
Genghiz | This is a backwards compatible change so I would recommend a backport as well. | 10:21:35 |
Genghiz | In reply to @genghiz:cdw.go7box.xyz Fwiw the tests work, but I’ll add a properly formatted review message in a bit so just merge post that I think. Done. | 10:35:26 |
Pol | Merged ~! | 10:59:16 |
Pol | * Merged ! | 10:59:17 |