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

Voting Power Contract Implementation #42

Merged
merged 12 commits into from
Mar 4, 2024
Merged

Conversation

matjazv
Copy link
Contributor

@matjazv matjazv commented Feb 16, 2024

What was the problem?

This PR resolves #40.

How was it solved?

  • Voting Power Contract was implemented based on the specs.
  • Deployment script was implemented.

How was it tested?

New unit tests were implemented.

@matjazv matjazv self-assigned this Feb 16, 2024
@matjazv matjazv force-pushed the 40-voting-power-contract branch 3 times, most recently from a287223 to f934245 Compare February 19, 2024 13:25
src/L2/L2VotingPower.sol Outdated Show resolved Hide resolved
src/L2/L2VotingPower.sol Outdated Show resolved Hide resolved
src/L2/L2VotingPower.sol Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Outdated Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Outdated Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Outdated Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Outdated Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Outdated Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Outdated Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Show resolved Hide resolved
src/L2/L2VotingPower.sol Outdated Show resolved Hide resolved
src/L2/L2VotingPower.sol Outdated Show resolved Hide resolved
src/L2/L2VotingPower.sol Outdated Show resolved Hide resolved
src/L2/L2VotingPower.sol Show resolved Hide resolved
test/L2/L2VotingPower.t.sol Outdated Show resolved Hide resolved
@matjazv
Copy link
Contributor Author

matjazv commented Feb 26, 2024

I needed to rebase this branch with main branch because branch was out-of-date.
Relevant commit for new changes: `Remove unused function, rename variable and fix Slither warnings.

@matjazv
Copy link
Contributor Author

matjazv commented Mar 4, 2024

force-push was due to rebasing with main branch.

@matjazv matjazv merged commit a7f6f65 into main Mar 4, 2024
3 checks passed
@matjazv matjazv deleted the 40-voting-power-contract branch March 4, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Voting Power Contract
3 participants