Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

no-simple-event-func-name rule is not case sensitive #12

Closed
davesag opened this issue Nov 27, 2017 · 1 comment
Closed

no-simple-event-func-name rule is not case sensitive #12

davesag opened this issue Nov 27, 2017 · 1 comment
Labels

Comments

@davesag
Copy link

davesag commented Nov 27, 2017

I am not sure if this is a bug or a feature but the no-simple-event-func-name rule does not see a difference between a function called claimSatisfied and an event ClaimSatisfied.

Is that rule supposed to be case insensitive?

@idrabenia
Copy link
Contributor

idrabenia commented Nov 28, 2017

Hi @davesag ,

Yes, there is a feature that related to this recommendations of ConsenSys Guide for Smart Contracts.

We understand that it may react on some standard contracts like ECR20 and made it the warning severity.

If execution of this rule is not helpful for you - it's possible to disable the rule in configuration file:

"rules": {
    ...
    "no-simple-event-func-name": false,
    ...
}

It's your own choice.

Thank you!
- Ilya

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants