Move some changeset to the "Removals" section of CHANGELOG (#4290)

Co-authored-by: Ernesto García <ernestognw@gmail.com> Co-authored-by: Francisco <fg@frang.io>
2 years ago · 2d1da295e6
parent 3902a410f1
commit 2d1da295e6
12 changed files with 24 additions and 57 deletions
--- a/.changeset/beige-ducks-flow.md
+++ b/.changeset/beige-ducks-flow.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove deprecated GovernorProposalThreshold module.
--- a/.changeset/dirty-mangos-sort.md
+++ b/.changeset/dirty-mangos-sort.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Removed presets in favor of [OpenZeppelin Contracts Wizard](https://wizard.openzeppelin.com/).
--- a/.changeset/fluffy-gifts-build.md
+++ b/.changeset/fluffy-gifts-build.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove ERC1820Implementer.
--- a/.changeset/friendly-suits-camp.md
+++ b/.changeset/friendly-suits-camp.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove Checkpoints.History.
--- a/.changeset/hungry-impalas-perform.md
+++ b/.changeset/hungry-impalas-perform.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-`ERC165Storage`: Removed this contract in favor of inheritance based approach. ([#3880](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3880))
--- a/.changeset/selfish-queens-rest.md
+++ b/.changeset/selfish-queens-rest.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove PullPayment and Escrow contracts (Escrow, ConditionalEscrow, RefundEscrow).
--- a/.changeset/spicy-ducks-cough.md
+++ b/.changeset/spicy-ducks-cough.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove the Timers library.
--- a/.changeset/swift-berries-sort.md
+++ b/.changeset/swift-berries-sort.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove ERC777 implementation.
--- a/.changeset/tame-geckos-search.md
+++ b/.changeset/tame-geckos-search.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove SafeMath and SignedSafeMath libraries.
--- a/.changeset/three-weeks-double.md
+++ b/.changeset/three-weeks-double.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': major
---
-
-Remove CrossChain contracts, including AccessControlCrossChain and all the vendored bridge interfaces.
--- a/.changeset/unlucky-snakes-drive.md
+++ b/.changeset/unlucky-snakes-drive.md
@ -1,5 +0,0 @@
---
-'openzeppelin-solidity': patch
---
-
-`Address`: Removed `isContract` because of its ambiguous nature and potential for misuse.
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@ -2,14 +2,36 @@

 ### Removals

-The following contracts and libraries were removed:
+The following contracts, libraries and functions were removed:

+- `Address.isContract` (because of its ambiguous nature and potential for misuse)
+- `Checkpoints.History`
 - `Counters`
 - `ERC20Snapshot`
 - `ERC20VotesComp`
+- `ERC165Storage` (in favor of inheritance based approach)
+- `ERC777`
+- `ERC1820Implementer`
 - `GovernorVotesComp`
+- `GovernorProposalThreshold` (deprecated since 4.4)
 - `PaymentSplitter`
- `TokenTimelock` (removed in favor of `VestingWallet`)
+- `PullPayment`
+- `SafeMath`
+- `SignedSafeMath`
+- `Timers`
+- `TokenTimelock` (in favor of `VestingWallet`)
+- All escrow contracts (`Escrow`, `ConditionalEscrow` and `RefundEscrow`)
+- All cross-chain contracts, including `AccessControlCrossChain` and all the vendored bridge interfaces
+- All presets in favor of [OpenZeppelin Contracts Wizard](https://wizard.openzeppelin.com/)
+
+These removals were implemented in the following PRs:
+
+- [3637](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3637)
+- [3880](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3880)
+- [3945](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3945)
+- [4258](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/4258)
+- [4276](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/4276)
+- [4289](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/4289)

 ### How to upgrade from 4.x