Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSE: support aliases for contract fields #653

Open
palinatolmach opened this issue Jun 26, 2024 · 0 comments
Open

CSE: support aliases for contract fields #653

palinatolmach opened this issue Jun 26, 2024 · 0 comments
Labels
cse engagement enhancement New feature or request

Comments

@palinatolmach
Copy link
Collaborator

palinatolmach commented Jun 26, 2024

We should add NatSpec annotations that allow to indicate that contract fields correspond to the same contract/account, e.g. /// @custom-kontrol-alias: escrowOne.oneEth, escrowTwo.oneEth,. They could then be used when CSEing contracts such as Governance:

contract Governance {
   OneEth oneEth;
   EscrowOne escrowOne;
   EscrowTwo escrowTwo;
   ...
}

contract EscrowOne {
   OneEth oneEth;
   ...
}

contract EscrowTwo {
   OneEth oneEth;
   ...
}

in order to only generate a single account for oneEth instead of three (one for each contract involved).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cse engagement enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant