Pourquoi pallet_collective<Instance2>?

Je vois cette cette instruction à plusieurs endroits dans le code :

pallet_collective::Config<Instance2>

Pourquoi pas Instance1 ?

parce que ça concerne les forgerons ?

Même si ça n’a pas d’importance c’est peut-être plus clair.

Ben non c’est le comité technique :face_with_raised_eyebrow:

git grep "pallet_collective::Config" $(git rev-list --all)

ne donne que pallet_collective::Config<Instance2>. Donc apparemment il n’y a jamais eu d’instance 1. Je croyais que treasury et technical committee avaient des instances différentes de collective, mais ce n’est pas le cas.

Ça remonte au commit 4fb19d805b72f0b8037ec9aa688b4e83f96c452f de elois en février 2022, il a probablement copié collé ça sans changer le numéro d’instance. On devrait pouvoir remettre 1 sans que ça change rien.

2 Likes

Peut-être dans l’idée que la taille décroît avec le numéro de l’instance, et qu’un jour on pourrait vouloir un collectif de tous les membres. La situation serait alors analogue à la TdC vs les forgerons.

Mais c’est arbitraire, pourquoi pas même créer Instance3 pour éviter toute confusion avec l’Instance2 utilisée pour la TdC. (et au passage, leur trouver des noms plus évocateurs)

2 Likes

OK merci pour vos réponses, je voulais surtout savoir s’il y avait une raison particulière que vous auriez en tête. On peut laisser comme c’est.

1 Like