Abolish interface contracts

[arwer13]

No description provided.

[folkyatina]

What do you think about merge two IStakingModule to one in commons/interfaces? We already have IEIP712.sol there.

[arwer13]

@folkyatina

What do you think about merge two IStakingModule to one in commons/interfaces? We already have IEIP712.sol there.

The problem is that IStakingModule has bytes calldata _calldata arg in requestValidatorsKeysForDeposits what makes it compilable either in 0.4.24 or 0.8.9.
We cannot either make common interface and inherit it in 0.4.24 and 0.8.9 defining requestValidatorsKeysForDeposits with bytes _calldata and with bytes calldata _calldata respectively.

[folkyatina]

@folkyatina

What do you think about merge two IStakingModule to one in commons/interfaces? We already have IEIP712.sol there.

The problem is that IStakingModule has bytes calldata _calldata arg in requestValidatorsKeysForDeposits what makes it compilable either in 0.4.24 or 0.8.9. We cannot either make common interface and inherit it in 0.4.24 and 0.8.9 defining requestValidatorsKeysForDeposits with bytes _calldata and with bytes calldata _calldata respectively.

In this case, I think, we should at least remove comments from 0.4.24, adding a notice about, where the docs can be found.