Persistent handling of issues about credo

[rrrene]

This is something I also thought about. It would be a non-trivial feature, so I am afraid this is more in the 1.0-territory.

So I absolutely agree with you. It's just that I always criticise it when other projects set the wrong priorities during the very early phases of their development and I don't want to be hypocrite.

Let's keep this issue open to remind me to put it on the roadmap! 👍

[maxnordlund]

Yeah, you're totally right, so maybe add a label longterm or something similar.

[rrrene]

@maxnordlund Credo v0.3.0 now includes support for a @lint attribute. These allow you to ignore indiviual checks on a perfunction basis. See CHANGELOG for further details.

What do you think?

[maxnordlund]

I love it 😸
This combined with a good commit message should suffice to close this. But I would like a flag to temporarily ignore the @lint attribute. Something one can run periodically to see if the ignores are still valid.

Close this if you like, unless you have a plan for this somehow 😉

[rrrene]

I like this idea of linting the @lint attributes, which could result in a message like

"XYCheck ignored via @lint, but XY does not report any issues here."

Great idea! 👍

[maxnordlund]

Yes, yes! Lint all the things. If it will work like that it could probably be on all the time instead of hidden behind a flag. Which would mean your linting ignores will never be grow stale.

Den 10 feb. 2016 12:56 skrev "René Föhring" [email protected]:

I like this idea of linting the @lint attributes, which could result in a
message like

"XYCheck ignored via @lint https://github.com/lint, but XY does not
report any issues here."

Great idea! [image: 👍]

—
Reply to this email directly or view it on GitHub
#5 (comment).

[rrrene]

Yeah, that's what I was thinking: a @lint checker that's just a regular check ... 😱

[chrismcg]

Yes, this is nice when you finally refactor some bit of code that needed ignored for a while and get to remove the @lint. It's also nice for "lint as you type" editor integration.

[TheFirstAvenger]

@rrrene This was created prior to the credo:disable feature, and I believe that is sufficient to close this issue?

[maxnordlund]

Yeah, I think so too.