| 24 Apr 2023 |
Pol | Yes, can't understand where it comes from | 19:20:09 |
Pol | From composer that's sure, but why ?! | 19:22:51 |
Pol | Found the issue. | 19:59:18 |
Pol | COMPOSER_ROOT_VERSION="${version}" \ | 20:03:59 |
Pol | https://github.com/NixOS/nixpkgs/blob/5ab2b1e0e1d9c3483371c93d20c08ecc1d1c4b5a/pkgs/development/php-packages/psalm/default.nix | 20:09:21 |
Pol | Now it's fixed. | 20:09:25 |
Pol | Amount of PHAR based software in Nix now: 1 (Composer) | 20:17:29 |
Pol | and we're back to green. | 20:37:39 |
Pol | Redacted or Malformed Event | 20:49:05 |
Pol | Redacted or Malformed Event | 20:50:47 |
Pol | Redacted or Malformed Event | 20:50:50 |
| 25 Apr 2023 |
Pol | Jan Tojnar: Anything else to do for the PHP Builder branch? | 08:20:20 |
Jan Tojnar | Pol: nothing comes to mind | 15:16:04 |
Pol | Good news :D | 15:17:40 |
tgerbet | I made a few comments on the PR, sorry for being late to the party 🫤 | 17:41:34 |
Pol | COMPOSER_HTACCESS_PROTECT WHaaaaaat ?! | 17:43:23 |
Pol | tgerbet: Got an even better idea while reading your comments. | 18:07:13 |
tgerbet | Great! Because to be honest I was not really a big fan of mine | 18:08:45 |
Pol |  Download image.png | 18:09:15 |
Pol | WDYT of that? | 18:09:18 |
Pol | Basically, I merge 2 composer.json files together to override. | 18:09:34 |
Pol | I think it's cleaner to do that. | 18:09:45 |
tgerbet | Yep indeed, it's more readable 👍️ | 18:10:45 |
Pol | We could even let users customize the overrides. | 18:10:58 |
Pol | But this could be done in a next phase. | 18:11:05 |
Pol | Regarding this: | 18:12:53 |
Pol |  Download image.png | 18:12:55 |
Pol | I don't know that thing yet. | 18:13:01 |
tgerbet | Yeah this is the hard part :/
Both options have trade-offs, I'm not sure if one is really better than the other or if there is another way to explore | 18:15:39 |
Pol | yeah it's scary. | 18:18:52 |