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

Recursively call extractCallsToSuper(methodName:) #1556

Merged
merged 1 commit into from
May 23, 2017
Merged

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented May 23, 2017

Fixes #1301.

@SwiftLintBot
Copy link

1 Warning
⚠️ Permanently added the RSA host key for IP address ‘192.30.253.112’ to the list of known hosts.
12 Messages
📖 Linting Aerial with this PR took 0.38s vs 0.38s on master (0% slower)
📖 Linting Alamofire with this PR took 2.46s vs 2.51s on master (1% faster)
📖 Linting Firefox with this PR took 11.09s vs 11.64s on master (4% faster)
📖 Linting Kickstarter with this PR took 17.9s vs 14.4s on master (24% slower)
📖 Linting Moya with this PR took 0.39s vs 0.33s on master (18% slower)
📖 Linting Nimble with this PR took 1.51s vs 1.55s on master (2% faster)
📖 Linting Quick with this PR took 0.55s vs 0.52s on master (5% slower)
📖 Linting Realm with this PR took 2.21s vs 2.48s on master (10% faster)
📖 Linting SourceKitten with this PR took 0.99s vs 1.06s on master (6% faster)
📖 Linting Sourcery with this PR took 2.38s vs 2.39s on master (0% faster)
📖 Linting Swift with this PR took 10.51s vs 10.96s on master (4% faster)
📖 Linting WordPress with this PR took 11.74s vs 11.79s on master (0% faster)

Generated by 🚫 Danger

jpsim added a commit that referenced this pull request May 23, 2017
@jpsim
Copy link
Collaborator Author

jpsim commented May 23, 2017

Local tests pass for SPM & Docker, and OSSCheck didn't report any newly introduced violations in OSS repos, merging.

@jpsim jpsim merged commit 118c73c into master May 23, 2017
@jpsim jpsim deleted the jp-recursive-super branch May 23, 2017 18:21
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.

3 participants