Remove deps, and modify contracts to support the change #1061
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
Issue: #929
Changes proposed in this pull request
Note: instead of copying the
SafeMath.sol
to the external directory in the governance, copied it to lib/external since other contracts use that are not in governance directory*Note: leaving
@openzeppelin/contracts-v4
as it is, if we rename it without the-v4
it breaks the beacon proxy and it's bytecode so the StakingPool addresses are calculated wrong *Test plan
Checklist
Review
When reviewing a PR, please indicate intention in comments using the following emojis: