| 9 Nov 2023 |
Ambroisie | Ah nice | 17:49:00 |
Ambroisie | J'avais pas vu ce draft, ça a l'air bien | 17:49:11 |
raitobezarius | je suis techniquement demi-mainteneur PG dans nixpkgs pour information | 17:49:34 |
raitobezarius | si jamais tu as besoin | 17:49:43 |
Ambroisie | Ça serait quoi la meilleure solution pour avoir le fix de mon package qui soit merge, même si le test est cassé à cause de Postgres
Je peux le faire utiliser SQLite en attendant, ça paraît être le mieux d'un point de vue "ne pas montrer d'option insecure à un utilisateur" | 17:50:40 |
Minijackson | In reply to @ambroisie:belanyi.fr Ça serait quoi la meilleure solution pour avoir le fix de mon package qui soit merge, même si le test est cassé à cause de Postgres Je peux le faire utiliser SQLite en attendant, ça paraît être le mieux d'un point de vue "ne pas montrer d'option insecure à un utilisateur" Est-ce que les données seront automatiquement migrées de Postgresql vers SQLite ? Ça pourrait impacter des utilisateurs sinon | 18:00:49 |
Ambroisie | C'est uniquement pour le test, le module ne touche pas aux options DB de l'utilisateur | 18:01:20 |
raitobezarius | utiliser SQLite me paraît mieux pour le moment mais bon c'est plus vraiment le même test quoi | 18:01:56 |
Ambroisie | Ouais c'est pas la DB recommandée, mais pour une installation perso'/le test isok | 18:03:15 |
Ambroisie | raitobezarius: je vois d'autres MRs qui commencent à pop-up à propos de l'issue, faudrait peut-être faire quelque chose pour éviter de merge des chose pas secure Exemple: https://github.com/NixOS/nixpkgs/pull/266511 | 21:44:50 |
raitobezarius | je remonte ça | 21:45:09 |
Ambroisie | Et sinon je suis plutôt d'accord avec une autre remarque que j'ai vu sur une MR: le choix de bump la version de Postgres pour la prochaine release a peut-être été fait un peu tard
Est-ce qu'il serait pas mieux dans le futur de le bump le plus tôt possible après le branch-off, comme ça on a un peu de temps de soak sur unstable ? | 22:11:12 |
| 10 Nov 2023 |
Julien | Je crois surtout que la PR a été merged sans le due diligence. Quand on mets à jour ce type de chose il fautse rendre compte que ça casse tout NixOS avant de merger. | 09:58:42 |
| 11 Nov 2023 |
| NinjaTrappeur left the room. | 07:14:38 |
| Picnoir joined the room. | 07:36:08 |
| Picnoir changed their profile picture. | 13:09:45 |
| 15 Nov 2023 |
| @grahamc:nixos.orgchanged room power levels. | 16:12:51 |
| @grahamc:nixos.org left the room. | 16:12:51 |
| @qyliss:fairydust.space joined the room. | 17:09:20 |
| @qyliss:fairydust.space | 17:11:39 |
| NixOS Moderation Botchanged room power levels. | 18:12:10 |