ZeppelinSolutions / solium-plugin-zeppelin

[DEPRECATED] Solium plugin for Zeppelin audits
MIT License
16 stars 10 forks source link

Errro validating natspec comments for functions #29

Closed jsperezg closed 5 years ago

jsperezg commented 6 years ago

When I don't add the comment /// @title ... for any of my contract functions the linter complains. However, with it added tests fail due to compile errors:

DocstringParsingError: Doc tag @title not valid for functions.

What i'm doing wrong?

mgregson commented 6 years ago

This looks fixed in https://github.com/OpenZeppelin/solium-plugin-zeppelin/commit/4a0698813fd0a9b73032fd69cf684c5510211454, but it hasn't been released yet.

Could we get this tagged and released?

come-maiz commented 6 years ago

Hey, I'm so surprised that there's people using this :scream: Where are you from? Are you finding it useful? Do you have feature requests?

I need to rename the package to the new solium name, so I've been delaying the release. But I promise to do it after the new three rules proposed in the pull requests are merged.

come-maiz commented 5 years ago

Hey @jsperezg and @mgregson. I'm sorry to inform you that I'm deprecating this package, so I'm closing this as won't fix. More info in the README.