Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

Vouching.create now requires the dependency address to be a contract. #225

Merged
merged 2 commits into from
Oct 10, 2018
Merged

Vouching.create now requires the dependency address to be a contract. #225

merged 2 commits into from
Oct 10, 2018

Conversation

nventuro
Copy link
Contributor

Fixes issue [zOS2V-O03] of the Nomic audit (part of #203).

Copy link
Contributor

@facuspagnuolo facuspagnuolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@facuspagnuolo facuspagnuolo added the status:ready-to-merge Order mergify to merge label Oct 10, 2018
@facuspagnuolo facuspagnuolo self-assigned this Oct 10, 2018
@facuspagnuolo
Copy link
Contributor

@nventuro failing vouching test, please review

@nventuro
Copy link
Contributor Author

Ha, that's what I get for not running tests after merging for conflicts.

@mergify mergify bot merged commit fb2a61e into OpenZeppelin:master Oct 10, 2018
@nventuro nventuro deleted the vouching-create-contract branch October 10, 2018 23:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:ready-to-merge Order mergify to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants