-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Update SourceKitten to 0.26.0 #2894
Conversation
Here's an example of your CHANGELOG entry: * Update SourceKitten to 0.26.0.
[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 |
@@ -195,15 +195,15 @@ def build(branch) | |||
perform("git fetch && git worktree add --detach #{dir} #{target}") | |||
end | |||
|
|||
Dir.chdir(dir) do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was this moved? This will slow down OSSCheck by preventing incremental builds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was getting build failures all the time IIRC
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unfortunately CI has deleted the build, but you can see that OSSCheck had failed before 956ea6f: https://github.com/realm/SwiftLint/runs/251739677
Fixes #2874?
Also should fix the CI failures on the
pod lib lint
.