feat(genesis): deployerAddress passed as parameter

[Villaquiranm]

closes #2573

Had to change the PR from a personal repository as the pipeline was failing
old PR: #2986

Contributors' checklist...

• Added new tests, or not needed, or not feasible
• Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
• Updated the official documentation or not needed
• No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
• Added references to related issues and PRs
• Provided any useful hints for running manual tests

[Gno2D2]

I'm a bot that assists the Gno Core team in maintaining this repository. My role is to ensure that contributors understand and follow our guidelines, helping to streamline the development process.

The following requirements must be fulfilled before a pull request can be merged.
Some requirement checks are automated and can be verified by the CI, while others need manual verification by a staff member.

These requirements are defined in this configuration file.

Automated Checks

🟢 Maintainers must be able to edit this pull request
🟢 The pull request head branch must be up-to-date with its base

Manual Checks

No manual checks match this pull request.

Debug

Automated Checks

Maintainers must be able to edit this pull request

If

🟢 Condition met
└── 🟢 On every pull request

Then

🟢 Requirement satisfied
└── 🟢 Maintainer can modify this pull request

The pull request head branch must be up-to-date with its base

If

🟢 Condition met
└── 🟢 On every pull request

Then

🟢 Requirement satisfied
└── 🟢 Head branch (Villaquiranm:feat/configure-address-deployer-genesis) is up to date with base (master): behind by 0 / ahead by 30

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

[zivkovicmilos]

Looks good 💯

I've left a few nitpicks, otherwise we are good to go 👍

We will need to change this functionality when / if we merge #3213

cc @Kouteki I'll open a new issue when #3213 is decided

[Villaquiranm]

Looks good 💯

I've left a few nitpicks, otherwise we are good to go 👍

We will need to change this functionality when / if we merge #3213

cc @Kouteki I'll open a new issue when #3213 is decided

Thanks for the review 🚀
I'll gladly tackle this if neccessary

[zivkovicmilos]

@Villaquiranm

Can you please unexport the deployerAddress field from the cfg struct, and make the defaultCreat a package global?
var defaultCreator = crypto.MustAddressFromString(...)

[Villaquiranm]

@Villaquiranm

Can you please unexport the deployerAddress field from the cfg struct, and make the defaultCreat a package global? var defaultCreator = crypto.MustAddressFromString(...)

@zivkovicmilos
Sorry I misunderstood you
changed here: 051cbd5

[jefft0]

Removed the review/triage-pending label because this PR was reviewed by core dev zivkovicmilos.