search

oasisprotocol / sapphire-paratime

The Sapphire ParaTime monorepo.
https://oasisprotocol.org/sapphire
Apache License 2.0
34 stars 24 forks source link

Remove OpenZeppelin Context #290

Closed aefhm closed 5 months ago

aefhm commented 5 months ago

Description

Our usage of Context is minimal.

netlify[bot] commented 5 months ago

Deploy Preview for oasisprotocol-sapphire-paratime canceled.

Name Link
Latest commit 46b0ee52e817b8b2fbc019293afbc6a9c37051bb
Latest deploy log https://app.netlify.com/sites/oasisprotocol-sapphire-paratime/deploys/65fb4e92766a8b0008209251
CedarMist commented 5 months ago

Given this was the only thing that required the openzeppelin dependency, surely we can remove that too?

aefhm commented 5 months ago

Given this was the only thing that required the openzeppelin dependency, surely we can remove that too?

We still have WrappedRose. https://github.com/oasisprotocol/sapphire-paratime/blob/36f00be823e9a246f67c99e79e0df721a16faf41/contracts/contracts/WrappedROSE.sol#L4-L5