-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Add option allow_redundancy to rule explicit_type_interface #2313
Conversation
Generated by 🚫 Danger |
Codecov Report
@@ Coverage Diff @@
## master #2313 +/- ##
=========================================
+ Coverage 91.99% 92% +<.01%
=========================================
Files 296 296
Lines 14886 14929 +43
=========================================
+ Hits 13694 13735 +41
- Misses 1192 1194 +2
Continue to review full report at Codecov.
|
I marked this PR [WIP] for now, as I need to update this to use a custom option following the discussions in #2312. |
289fe84
to
7643c8b
Compare
Just added an option like discussed in #2312 – this is now ready to merge from my point of view. Looking forward to for reviews! |
CHANGELOG.md
Outdated
@@ -67,6 +67,11 @@ | |||
[Marcelo Fabri](https://github.com/marcelofabri) | |||
[#2233](https://github.com/realm/SwiftLint/issues/2233) | |||
|
|||
* Make rule `explicit_type_interface` compatible with rule |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you move this to the "Enhancements" section and mention the configuration option?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, just did.
7643c8b
to
b4f960b
Compare
I just rebased and moved the changelog entry accordingly. |
@@ -35,6 +35,11 @@ | |||
* Add `private_only` configuration to `prefixed_toplevel_constant` rule | |||
[Keith Smiley](https://github.com/keith) | |||
[#2315](https://github.com/realm/SwiftLint/pull/2315) | |||
|
|||
* Make rule `explicit_type_interface` compatible with rule |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please mention that this is possible with the allow_redundancy
configuration key?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Fixes #2312.