Skip to content
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 excluded_match_kinds custom rule config parameter #3336

Merged
merged 8 commits into from
Sep 11, 2020
Merged

Add excluded_match_kinds custom rule config parameter #3336

merged 8 commits into from
Sep 11, 2020

Conversation

ryandemo
Copy link
Contributor

This allows custom rules to define an excluded_match_kinds array instead of listing out all but one or a few of the SyntaxKinds in match_kinds. Rules that include both match_kinds and excluded_match_kinds will be invalid, since there's not an obvious way to resolve the two without an arbitrary priority between them.

@SwiftLintBot
Copy link

SwiftLintBot commented Sep 10, 2020

12 Messages
📖 Linting Aerial with this PR took 1.85s vs 1.81s on master (2% slower)
📖 Linting Alamofire with this PR took 2.69s vs 2.68s on master (0% slower)
📖 Linting Firefox with this PR took 9.25s vs 9.19s on master (0% slower)
📖 Linting Kickstarter with this PR took 15.71s vs 15.64s on master (0% slower)
📖 Linting Moya with this PR took 1.37s vs 1.35s on master (1% slower)
📖 Linting Nimble with this PR took 1.37s vs 1.37s on master (0% slower)
📖 Linting Quick with this PR took 0.64s vs 0.64s on master (0% slower)
📖 Linting Realm with this PR took 2.68s vs 2.72s on master (1% faster)
📖 Linting SourceKitten with this PR took 1.03s vs 1.04s on master (0% faster)
📖 Linting Sourcery with this PR took 7.11s vs 7.07s on master (0% slower)
📖 Linting Swift with this PR took 10.84s vs 10.79s on master (0% slower)
📖 Linting WordPress with this PR took 16.87s vs 16.86s on master (0% slower)

Generated by 🚫 Danger

@codecov-commenter
Copy link

codecov-commenter commented Sep 10, 2020

Codecov Report

Merging #3336 into master will increase coverage by 0.01%.
The diff coverage is 98.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3336      +/-   ##
==========================================
+ Coverage   90.56%   90.58%   +0.01%     
==========================================
  Files         416      416              
  Lines       20357    20406      +49     
==========================================
+ Hits        18437    18485      +48     
- Misses       1920     1921       +1     
Impacted Files Coverage Δ
...sts/SwiftLintFrameworkTests/CustomRulesTests.swift 96.89% <97.36%> (+0.09%) ⬆️
.../Rules/RuleConfigurations/RegexConfiguration.swift 93.54% <100.00%> (+1.71%) ⬆️
...e/SwiftLintFramework/Rules/Style/CustomRules.swift 98.18% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4083f16...bb8f34c. Read the comment docs.

Copy link
Collaborator

@jpsim jpsim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Will merge once CI passes.

@jpsim jpsim merged commit 51544bb into realm:master Sep 11, 2020
@ryandemo ryandemo deleted the excluded_match_kinds branch September 11, 2020 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants