-
Notifications
You must be signed in to change notification settings - Fork 87
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
There's no ignore option for the state_record_and_type
rule
#551
Comments
Have you tried adding the following attribute to your module(s)? -elvis([{elvis_style, state_record_and_type, disable}]). |
Now I tried, but still get the error. This is my source file:
|
I have to add I'm using |
I can confirm this is a bug. Thanks for reporting, @NAR |
@paulo-ferraz-oliveira / @Licenser : Can it be that |
Yeah… I think that's it! We need a new release of |
Yes, I would say that In the meantime could you check if the behavior persists by forcing (via either |
This is correct. Support for both the directive and the generic ignore is not released (or published to Hex.pm) in either |
Also related: check https://github.com/inaka/elvis_core/blob/master/test/examples/pass_state_record_and_type_elvis_attr.erl (what @elbrujohalcon was referring to). |
With the latest
line suppresses the warning, I can confirm that. I could only test with |
Alright… I'll see if I can cut a release of |
Aaaaand… project-fifo/rebar3_lint#19 |
@NAR, I'm closing this since it's only a matter of releasing |
It is not possible to ignore the warnings from the
state_record_and_type
rule, the code does not check for theignore
option. It would be really cool if I could suppress this warning :-)The text was updated successfully, but these errors were encountered: