Skip to content

Conversation

@dmx974
Copy link
Contributor

@dmx974 dmx974 commented Sep 16, 2017

Best practice for lisibility.

@SylTi
Copy link
Contributor

SylTi commented Sep 18, 2017

I agree with you.
There is a bunch of files where they are missing.
Maybe it's better to update all of them in a single commit/pull request?

@dmx974
Copy link
Contributor Author

dmx974 commented Sep 18, 2017 via email

@SylTi
Copy link
Contributor

SylTi commented Sep 18, 2017

nice!
https://github.com/OpenZeppelin/zeppelin-solidity/pull/448/files
Already did the RefundVault there.
Very nice to see a french compatriot :D

@dmx974
Copy link
Contributor Author

dmx974 commented Sep 18, 2017

Here we go:
#456

;-)

@frangio
Copy link
Contributor

frangio commented Sep 18, 2017

Closing in favor of #456.

@frangio frangio closed this Sep 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants