create-testnet-data's implementation: explain why drepDelegs can be left empty

smelc · smelc · commit a12df9d34397 · 2024-12-11T14:47:35.000+01:00
diff --git a/cardano-cli/src/Cardano/CLI/EraBased/Run/Genesis/CreateTestnetData.hs b/cardano-cli/src/Cardano/CLI/EraBased/Run/Genesis/CreateTestnetData.hs
@@ -516,7 +516,9 @@ runGenesisCreateTestNetDataCmd
                     { L.drepExpiry = EpochNo 1_000
                     , L.drepAnchor = SNothing
                     , L.drepDeposit = max (L.Coin 1_000_000) minDeposit
-                    , L.drepDelegs = Set.empty
+                    , L.drepDelegs = Set.empty -- We don't need to populate this field (field "initialDReps"."keyHash-*"."delegators" in the JSON)
+                    -- because its content is derived from the "delegs" field ("cgDelegs" above)
+                    -- More context is provided here: https://github.com/IntersectMBO/cardano-cli/pull/987
                     }
                 )
             )

Original file line number	Diff line number	Diff line change
`@@ -516,7 +516,9 @@ runGenesisCreateTestNetDataCmd`
`516`	`516`	`{ L.drepExpiry = EpochNo 1_000`
`517`	`517`	`, L.drepAnchor = SNothing`
`518`	`518`	`, L.drepDeposit = max (L.Coin 1_000_000) minDeposit`
`519`		`- , L.drepDelegs = Set.empty`
	`519`	`+ , L.drepDelegs = Set.empty -- We don't need to populate this field (field "initialDReps"."keyHash-*"."delegators" in the JSON)`
	`520`	`+ -- because its content is derived from the "delegs" field ("cgDelegs" above)`
	`521`	`+ -- More context is provided here: https://github.com/IntersectMBO/cardano-cli/pull/987`
`520`	`522`	`}`
`521`	`523`	`)`
`522`	`524`	`)`