| 19 Mar 2023 |
 Lars Strojny | See you!
| 22:19:53 |
| Lars Strojny | Btw, changing pkgs to system gives me that | 22:20:01 |
| Lars Strojny |
at /nix/store/s8427y5ln38zyyya2vgh3hfk0cq414mi-source/src/phps.nix:113:12:
112| if builtins.isString php
113| then pkgs."${php}"
| ^
114| else php;```
| 22:20:05 |
 Pol | `nix flake update` :) | 22:20:25 |
| Lars Strojny | :-D | 22:20:40 |
| 20 Mar 2023 |
| Pol | Lars Strojny: I just submitted a PR to refactor your flake.nix: https://github.com/lstrojny/uffff/pull/11 | 07:36:40 |
| Pol | It's purely informative and it's totally fine to reject it :) | 07:36:57 |
| Pol | Ok since ryantm-bot is not working for PHP and Composer, I created the PR: https://github.com/NixOS/nixpkgs/pull/222253 | 21:36:35 |
| Pol | Please have a look on it :) | 21:36:51 |
| Pol | Have a good night | 21:36:54 |
| 21 Mar 2023 |
| Pol | Job done etu https://github.com/NixOS/nixpkgs/pull/222313 | 08:04:57 |
| Pol | A PHP related PR: https://github.com/NixOS/nixpkgs/pull/221985 | 12:42:41 |
| Lars Strojny | In reply to @drupol:matrix.org
Lars Strojny: I just submitted a PR to refactor your flake.nix: https://github.com/lstrojny/uffff/pull/11 Btw: needed to revise it slightly again so that the fork is used as an input: https://github.com/lstrojny/uffff/commit/05046778c1285f9b4cd9793f76be9824e196fe77 | 19:36:38 |
| Pol | sure sure :) | 19:37:08 |
| Pol | Don't forget to run nix fmt at the end :) | 19:37:16 |
| Lars Strojny | my vim setup does that for me | 19:40:36 |
| Lars Strojny | Just that I have a wider line width configured | 19:40:44 |
| Pol | Ok you already have a formatter then | 19:43:35 |
| Pol | Oooh you changed to nixfmt | 19:44:08 |
| Pol | I missed that | 19:44:10 |
| Pol | All good then :) | 19:44:13 |
| 23 Mar 2023 |
| Pol | Are you ok to merge this? https://github.com/NixOS/nixpkgs/pull/221845 | 07:53:20 |
| Lars Strojny | In reply to @drupol:matrix.org
Are you ok to merge this? https://github.com/NixOS/nixpkgs/pull/221845 If you ask me, yes. Discussed the issue off band with Jan and he was also fine | 12:55:48 |
| Pol | yep, it was merely to let other maintainers know about the PR :) | 13:04:02 |
| Pol | I'm also ok to merge it | 13:04:07 |
| 27 Mar 2023 |
| Pol | PHPBench, init at 1.2.10 : https://github.com/NixOS/nixpkgs/pull/223365 Ready for review! | 09:27:44 |
| 28 Mar 2023 |
| Pol | ^^ | 06:21:33 |
 Gaël Reyrol | I know PHP 7.4 is deprecated but PHPBench is still supporting it, is it on purpose you decided to directly use PHP 8.0 ? | 07:40:48 |
 @etu:failar.nu | We don't have php74 in nixpkgs | 08:05:52 |
| @etu:failar.nu | The reason we drop it right after the XX.11 release is that it gives time to figure out everything that breaks of the removal before the XX.05 release 🙂 | 08:07:06 |
