some improvements in config files

[pcarranzav]

No description provided.

[codecov]

Codecov Report

Base: 91.51% // Head: 91.51% // No change to project coverage 👍

Coverage data is based on head (ffdca3b) compared to base (3f6c55e).
Patch coverage: 99.08% of modified lines in pull request are covered.

❗ Current head ffdca3b differs from pull request most recent head 80ef55a. Consider uploading reports for the commit 80ef55a to get more accurate results

Additional details and impacted files

@@              Coverage Diff               @@
##           pcv/l2-bridge     #679   +/-   ##
==============================================
  Coverage          91.51%   91.51%           
==============================================
  Files                 41       41           
  Lines               1980     1980           
  Branches             346      346           
==============================================
  Hits                1812     1812           
  Misses               168      168           

Flag	Coverage Δ
unittests	91.51% <99.08%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
contracts/governance/Managed.sol	94.54% <50.00%> (ø)
contracts/gateway/BridgeEscrow.sol	100.00% <100.00%> (ø)
contracts/gateway/GraphTokenGateway.sol	100.00% <100.00%> (ø)
contracts/gateway/L1GraphTokenGateway.sol	100.00% <100.00%> (ø)
contracts/l2/gateway/L2GraphTokenGateway.sol	100.00% <100.00%> (ø)
contracts/l2/token/GraphTokenUpgradeable.sol	100.00% <100.00%> (ø)
contracts/l2/token/L2GraphToken.sol	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[abarmat]

Can you remind me why we removed the L2 bridge to be a minter in the fixture?

[pcarranzav]

Because we are already using the L2GraphToken.setGateway function to add it as the gateway minter, that can mint and burn through the bridgeMint and bridgeBurn functions.

[tmigone]

Small nit with theL2GraphToken config. It can be simplified from:

  L2GraphToken:
    proxy: true
    init:
      owner: "${{Env.deployer}}"
    calls:
      - fn: "renounceMinter"
      - fn: "transferOwnership"
        owner: *governor

to this:

  L2GraphToken:
    proxy: true
    init:
      owner: *governor