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

Analyze all files listed in the command even if the --path option is used #3951

Merged
merged 1 commit into from
Apr 15, 2022

Conversation

SimplyDanny
Copy link
Collaborator

Same change like #3923 for the analyze command.

@SimplyDanny SimplyDanny enabled auto-merge (squash) April 15, 2022 13:07
@SwiftLintBot
Copy link

SwiftLintBot commented Apr 15, 2022

12 Messages
📖 Linting Aerial with this PR took 0.98s vs 0.98s on master (0% slower)
📖 Linting Alamofire with this PR took 1.11s vs 1.11s on master (0% slower)
📖 Linting Firefox with this PR took 4.19s vs 4.21s on master (0% faster)
📖 Linting Kickstarter with this PR took 7.46s vs 7.47s on master (0% faster)
📖 Linting Moya with this PR took 4.37s vs 4.44s on master (1% faster)
📖 Linting Nimble with this PR took 0.4s vs 0.4s on master (0% slower)
📖 Linting Quick with this PR took 0.18s vs 0.18s on master (0% slower)
📖 Linting Realm with this PR took 10.22s vs 9.93s on master (2% slower)
📖 Linting SourceKitten with this PR took 0.33s vs 0.33s on master (0% slower)
📖 Linting Sourcery with this PR took 2.05s vs 2.02s on master (1% slower)
📖 Linting Swift with this PR took 3.22s vs 3.22s on master (0% slower)
📖 Linting WordPress with this PR took 7.54s vs 7.54s on master (0% slower)

Generated by 🚫 Danger

@SimplyDanny SimplyDanny merged commit 291b500 into realm:master Apr 15, 2022
@SimplyDanny SimplyDanny deleted the analyze-all-files branch April 15, 2022 15:08
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