-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
enabled_rules option in .swiftlint.yml? #256
Labels
enhancement
Ideas for improvements of existing features and rules.
Comments
Thanks for the suggestion! I think we should add both an
|
@jpsim I think it starts to get complicated if we allow blacks lists, black lists with opt in, and white lists. Why not just got with the "either black list or white list" approach, similar to |
You're right @scottrhoyt. Re-reading my comment I don't recall why I though that would be a good idea. |
Merged
done in #436. |
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
enabled_rules
option in the.swiftlint.yml
config file that works as a whitelist.disabled_rules
option, each time I upgrade the swift lint binary, and there have been new rules added, I have to disable more rules (otherwise swiftlint will generate warnings / errors for rules I don't necessarily want).The text was updated successfully, but these errors were encountered: