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 AutomaticTestableRule conformance to EnumCaseAssociatedValuesLengthRule #3005

Merged
merged 1 commit into from
Jan 3, 2020

Conversation

marcelofabri
Copy link
Collaborator

Follow up from #2998

@marcelofabri marcelofabri merged commit fbbd3fc into master Jan 3, 2020
@marcelofabri marcelofabri deleted the mf-automatic-testable-enum-case-count branch January 3, 2020 08:34
@SwiftLintBot
Copy link

1 Warning
⚠️ Please include a CHANGELOG entry to credit yourself!
You can find it at CHANGELOG.md.
12 Messages
📖 Linting Aerial with this PR took 1.18s vs 1.18s on master (0% slower)
📖 Linting Alamofire with this PR took 2.21s vs 2.21s on master (0% slower)
📖 Linting Firefox with this PR took 8.86s vs 8.85s on master (0% slower)
📖 Linting Kickstarter with this PR took 14.43s vs 14.49s on master (0% faster)
📖 Linting Moya with this PR took 1.13s vs 1.17s on master (3% faster)
📖 Linting Nimble with this PR took 1.37s vs 1.37s on master (0% slower)
📖 Linting Quick with this PR took 0.53s vs 0.53s on master (0% slower)
📖 Linting Realm with this PR took 2.34s vs 2.39s on master (2% faster)
📖 Linting SourceKitten with this PR took 1.01s vs 0.98s on master (3% slower)
📖 Linting Sourcery with this PR took 6.71s vs 6.69s on master (0% slower)
📖 Linting Swift with this PR took 12.25s vs 12.23s on master (0% slower)
📖 Linting WordPress with this PR took 14.56s vs 14.59s on master (0% faster)

Here's an example of your CHANGELOG entry:

* Add AutomaticTestableRule conformance to EnumCaseAssociatedValuesLengthRule.  
  [marcelofabri](https://github.com/marcelofabri)
  [#issue_number](https://github.com/realm/SwiftLint/issues/issue_number)

note: There are two invisible spaces after the entry's text.

Generated by 🚫 Danger

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.

2 participants