add_metis_calendars_to_linkal #8

Closed
mdebray wants to merge 1 commit from add_metis_calendars_to_linkal into main
Member

Bon le script jq est moche. On pourrait le remplacer par du nix. Normalement ça ~marche (j'ai pas pu tester complètement dans la vm à cause d'histoires de réseau et de dns)

Bon le script jq est moche. On pourrait le remplacer par du nix. Normalement ça ~marche (j'ai pas pu tester complètement dans la vm à cause d'histoires de réseau et de dns)
mdebray added 1 commit 2023-10-03 01:14:31 +02:00
mdebray force-pushed add_metis_calendars_to_linkal from b9c8214803 to 0ee8760f8d 2023-10-03 01:18:09 +02:00 Compare
thubrecht reviewed 2023-10-03 09:38:29 +02:00
@ -2,3 +2,3 @@
let
inherit (lib) mapAttrs' mkEnableOption mkIf mkOption nameValuePair types;
inherit (lib) mapAttrs mapAttrs' mkEnableOption mkIf mkOption nameValuePair types;
Owner

mapAttrs est un builtin depuis nix 2.1, en plus t'as pas l'air de l'utiliser ^^

`mapAttrs` est un builtin depuis nix 2.1, en plus t'as pas l'air de l'utiliser ^^
Author
Member

Effectivement

Effectivement
mdebray marked this conversation as resolved
thubrecht reviewed 2023-10-03 09:38:31 +02:00
@ -28,0 +28,4 @@
config = mkOption {
type = path;
default = jsonFormat.generate "linkal-config.json" { inherit (config) calendars; };
internal = true;
Owner

Pourquoi internal si l'objectif c'est de pouvoir override le fichier de config ?

Pourquoi internal si l'objectif c'est de pouvoir override le fichier de config ?
Author
Member

Hmm l'idée était de dire c'est une interface uniquement pour les utilisateurs avancés mais en vrai on peut l'enlever

Hmm l'idée était de dire c'est une interface uniquement pour les utilisateurs avancés mais en vrai on peut l'enlever
thubrecht reviewed 2023-10-03 09:40:13 +02:00
@ -32,6 +25,11 @@ let
};
in {
_module.args.metisProviders = {
Owner

Hmmm, je suis pas certain que de passer metisProvider à tous les modules soit une super idée, ce serait plus propre de laisser metis definir ses providers et de les récupérer avec (import sources.metis { ... }).providers

Hmmm, je suis pas certain que de passer `metisProvider` à tous les modules soit une super idée, ce serait plus propre de laisser metis definir ses providers et de les récupérer avec `(import sources.metis { ... }).providers`
Author
Member

dans ce cas autant faire en nix et pas en jq l'adaptation du fichier de config

dans ce cas autant faire en nix et pas en jq l'adaptation du fichier de config
Author
Member

Bon étant donné que @thubrecht tu es favorable à la méthode de "On s'en fout des IFD" je fait recommencer from scratch

Bon étant donné que @thubrecht tu es favorable à la méthode de "On s'en fout des IFD" je fait recommencer from scratch
mdebray closed this pull request 2023-10-20 13:01:15 +02:00
mdebray deleted branch add_metis_calendars_to_linkal 2023-10-23 11:36:36 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/infrastructure#8
No description provided.