Allowing to ignore modules on Credo.Check.Readability.AliasAs

Open kelvinst opened this issue 3 years ago • 4 comments

Closes https://github.com/rrrene/credo-proposals/issues/81

This PR is introducing a new :ignore param on Credo.Check.Readability.AliasAs, to allow users to configure a list of modules that should be ignored by this check.

For example: with {Credo.Check.Readability.AliasAs, ignore: [Test]} on the config, the check would not warn for alias Test, as: T anywhere.

Aug 06 '22 16:08 kelvinst

Thanks for the thorough review @rrrene. Will work on your suggestions soon!

Sep 02 '22 17:09 kelvinst

Checking in: Any developments?

Dec 17 '23 21:12 rrrene

Checking in: Any news?

Jan 30 '24 05:01 rrrene

Sorry for the delay. I finally am getting back on tracks after a huge hiatus in contributions. I'm focusing on this first, so I'll get back to you soon.

Feb 12 '24 12:02 kelvinst