!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

73 Members
A room for PHP developers running on Nix22 Servers

Load older messages


SenderMessageTime
23 Sep 2024
@drupol:matrix.orgPolhttps://github.com/NixOS/nixpkgs/pull/34390007:21:21
@drupol:matrix.orgPol patka: Yeah :) 07:21:30
@drupol:matrix.orgPol patka: I've also fixed this: https://github.com/NixOS/nixpkgs/pull/343900/commits/3bcaabbb7119cb06f7f79770baf2e11ad293955c 07:21:45
@patka:envs.net@patka:envs.netOopsie, thanks!07:23:59
@patka:envs.net@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@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:cdw.go7box.xyzGenghiz
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
@drupol:matrix.orgPolQuick question, how do you usually test PR ? Do you cherry pick the commit in your branch or do another trick ?07:32:17
@genghiz:cdw.go7box.xyzGenghiz
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
@drupol:matrix.orgPolBut in this case, you need to test this against fireflii, how do you do then?09:37:58
@genghiz:cdw.go7box.xyzGenghiz

Nixpkgs-review pr -p firefly-iii

09:38:22
@drupol:matrix.orgPolnice09:39:29
@drupol:matrix.orgPolI just notice a small typo09:39:35
@drupol:matrix.orgPolI will fix it after that PR is merged.09:39:42
@drupol:matrix.orgPol setComposeRootVersion -> setComposerRootVersion 09:40:01
@drupol:matrix.orgPol setComposeEnvVariables -> setComposerEnvVariables 09:40:12
@drupol:matrix.orgPolLet me do it within this PR09:40:23
@genghiz:cdw.go7box.xyzGenghizSo firefly-iii seems to be fixed. 09:41:06
@drupol:matrix.orgPolDone, I just fixed the typo09:41:28
@genghiz:cdw.go7box.xyzGenghizOnce you merge I’ll close the firefly-iii fix PR. 09:41:29
@drupol:matrix.orgPolCan you try it again?09:41:40
@drupol:matrix.orgPolAnd post some feedback in the PR ?09:41:45
@genghiz:cdw.go7box.xyzGenghizSame PR number?09:41:48
@drupol:matrix.orgPolyeap09:41:54
@genghiz:cdw.go7box.xyzGenghiz2 mins09:42:10
@genghiz:cdw.go7box.xyzGenghizFwiw 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:cdw.go7box.xyzGenghizThis is a backwards compatible change so I would recommend a backport as well. 10:21:35
@genghiz:cdw.go7box.xyzGenghiz
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
@drupol:matrix.orgPolMerged ~!10:59:16
@drupol:matrix.orgPol * Merged !10:59:17

Show newer messages


Back to Room ListRoom Version: 6