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

Remove code in preparation for v5.0 #4258

Merged
merged 37 commits into from
May 19, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
cd07e1b
Remove cross-chain contracts
Amxx May 16, 2023
42b3aef
Remove Escrow & PullPayment contracts
Amxx May 16, 2023
ff68a57
Remove ERC777
Amxx May 16, 2023
18e63a7
Update documentation's nav
Amxx May 16, 2023
64bf67c
add changeset
Amxx May 16, 2023
aeb12fa
Remove the Timer libary
Amxx May 16, 2023
0ea8756
Remove deprecated draft-xxx files.
Amxx May 16, 2023
3344612
Remove getters with error strings
Amxx May 16, 2023
5624e52
Remove SafeERC20.safeApprove
Amxx May 16, 2023
0a3ccc2
Remove AccessControl._setupRole
Amxx May 16, 2023
f3e4e14
Remove deprecated getters in Proxy & deprecated error code in ECDSA
Amxx May 16, 2023
a06d55f
Remove GovernorProposalThreshold
Amxx May 16, 2023
a949b62
Remove ERC1820Implementer
Amxx May 16, 2023
6831f92
fix lint
Amxx May 16, 2023
7b385c6
remove tests for ERC1820Implementer
Amxx May 16, 2023
cd0407f
fix test/urils/Create2.test.js dependency on ERC1820Implementer
Amxx May 16, 2023
edc27fe
remove deprecated and duplicate storage
Amxx May 16, 2023
69c5dee
Remove Checkpoints.History
Amxx May 17, 2023
c974e7f
Remove SafeMath.sol and move tryXxx function to Math.sol
Amxx May 17, 2023
d39ab48
re-add PullPayment (with immutable ownership of the Escrow)
Amxx May 17, 2023
b2be20e
Revert "re-add PullPayment (with immutable ownership of the Escrow)"
Amxx May 17, 2023
f714926
Update "_Available since" for tryXxx operations
Amxx May 19, 2023
15639d7
Apply suggestions from code review
Amxx May 19, 2023
e519bc3
Update Math.test.js
Amxx May 19, 2023
2cd0d41
cleanup EnumerableMap.test.js
Amxx May 19, 2023
815bf2a
cleanup Checkpoints tests
Amxx May 19, 2023
a19eb6a
retreive checkpoint sizes from the generation scripts
Amxx May 19, 2023
d0e778f
fix foundry test
Amxx May 19, 2023
932ae59
Fix upgradeable.patch
Amxx May 19, 2023
b9991cc
Update .changeset/selfish-queens-rest.md
Amxx May 19, 2023
4d11483
add missing await
frangio May 19, 2023
32b6b60
Remove SignedSafeMath library and tests
Amxx May 19, 2023
e21211b
remove mention to SafeMath in SafeCast
Amxx May 19, 2023
9456fce
Update test/utils/Create2.test.js
Amxx May 19, 2023
2d2e8fa
Update test/utils/Checkpoints.test.js
frangio May 19, 2023
9193cea
remove mention of safemath from template
frangio May 19, 2023
41732d3
remove ERC-1820 docs and move interfaces
frangio May 19, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/beige-ducks-flow.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove deprecated GovernorProposalThreshold module.
5 changes: 5 additions & 0 deletions .changeset/fluffy-gifts-build.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove ERC1820Implementer.
5 changes: 5 additions & 0 deletions .changeset/friendly-suits-camp.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove Checkpoints.History.
5 changes: 5 additions & 0 deletions .changeset/selfish-queens-rest.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove PullPayment and Escrow contracts (Escrow, ConditionalEscrow, RefundEscrow).
5 changes: 5 additions & 0 deletions .changeset/spicy-ducks-cough.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove the Timers library.
5 changes: 5 additions & 0 deletions .changeset/swift-berries-sort.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove ERC777 implementation.
5 changes: 5 additions & 0 deletions .changeset/tame-geckos-search.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove SafeMath and SignedSafeMath libraries.
5 changes: 5 additions & 0 deletions .changeset/three-weeks-double.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'openzeppelin-solidity': major
---

Remove CrossChain contracts, including AccessControlCrossChain and all the vendored bridge interfaces.
24 changes: 0 additions & 24 deletions contracts/access/AccessControl.sol
Original file line number Diff line number Diff line change
Expand Up @@ -183,30 +183,6 @@ abstract contract AccessControl is Context, IAccessControl, ERC165 {
_revokeRole(role, account);
}

/**
* @dev Grants `role` to `account`.
*
* If `account` had not been already granted `role`, emits a {RoleGranted}
* event. Note that unlike {grantRole}, this function doesn't perform any
* checks on the calling account.
*
* May emit a {RoleGranted} event.
*
* [WARNING]
* ====
* This function should only be called from the constructor when setting
* up the initial roles for the system.
*
* Using this function in any other way is effectively circumventing the admin
* system imposed by {AccessControl}.
* ====
*
* NOTE: This function is deprecated in favor of {_grantRole}.
*/
function _setupRole(bytes32 role, address account) internal virtual {
_grantRole(role, account);
}

/**
* @dev Sets `adminRole` as ``role``'s admin role.
*
Expand Down
45 changes: 0 additions & 45 deletions contracts/access/AccessControlCrossChain.sol

This file was deleted.

2 changes: 0 additions & 2 deletions contracts/access/README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@ This directory provides ways to restrict who can access the functions of a contr

{{AccessControl}}

{{AccessControlCrossChain}}

{{IAccessControlEnumerable}}

{{AccessControlEnumerable}}
Expand Down
54 changes: 0 additions & 54 deletions contracts/crosschain/CrossChainEnabled.sol

This file was deleted.

34 changes: 0 additions & 34 deletions contracts/crosschain/README.adoc

This file was deleted.

49 changes: 0 additions & 49 deletions contracts/crosschain/amb/CrossChainEnabledAMB.sol

This file was deleted.

35 changes: 0 additions & 35 deletions contracts/crosschain/amb/LibAMB.sol

This file was deleted.

44 changes: 0 additions & 44 deletions contracts/crosschain/arbitrum/CrossChainEnabledArbitrumL1.sol

This file was deleted.

40 changes: 0 additions & 40 deletions contracts/crosschain/arbitrum/CrossChainEnabledArbitrumL2.sol

This file was deleted.

Loading