Skip to content
This repository was archived by the owner on Jun 7, 2020. It is now read-only.

Conversation

@rafaelks
Copy link
Contributor

@rafaelks rafaelks commented Nov 8, 2018

@RocketChat/ios

Updates

  • Crashlytics from 3.10.7 to 3.11.1CHANGELOG
  • Fabric from 1.7.11 to 1.8.1CHANGELOG
  • Firebase from 5.3.0 to 5.12.0CHANGELOG
  • Nuke from 7.3.2 to 7.5.1CHANGELOG
  • Nuke-FLAnimatedImage-Plugin from 5.0 to 6.0CHANGELOG
  • Realm from 3.9.0 to 3.11.1CHANGELOG
  • SwiftyJSON from 4.1.0 to 4.2.0CHANGELOG

@rafaelks rafaelks added this to the 3.2.0 milestone Nov 8, 2018
public init(urlRequest: URLRequest, targetSize: CGSize, contentMode: ImageDecompressor.ContentMode) {
self = ImageRequest(urlRequest: urlRequest)
self.processor = AnyImageProcessor(ImageDecompressor(targetSize: targetSize, contentMode: contentMode))
public init(urlRequest: URLRequest, targetSize: CGSize, contentMode: ImageDecompressor.ContentMode, upscale: Bool = false) {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 128 characters (line_length)

}

internal func decompress(_ image: UIImage, targetSize: CGSize, contentMode: ImageDecompressor.ContentMode) -> UIImage {
internal func decompress(_ image: UIImage, targetSize: CGSize, contentMode: ImageDecompressor.ContentMode, upscale: Bool) -> UIImage {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 134 characters (line_length)

/// - parameter `maxConcurrentRequestCount`: 2 by default.
public init(pipeline: ImagePipeline = ImagePipeline.shared, maxConcurrentRequestCount: Int = 2) {
/// - parameter destination: `.memoryCache` by default.
public init(pipeline: ImagePipeline = ImagePipeline.shared, destination: Destination = .memoryCache, maxConcurrentRequestCount: Int = 2) {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 142 characters (line_length)

}

private func _url(for key: Key) -> URL? {
/* testable */ func _url(for key: Key) -> URL? {

Choose a reason for hiding this comment

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

Identifier Name Violation: Function name should start with a lowercase character: '_url(for:)' (identifier_name)

@codecov
Copy link

codecov bot commented Nov 8, 2018

Codecov Report

Merging #2292 into develop will increase coverage by 0.1%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop    #2292     +/-   ##
==========================================
+ Coverage    25.86%   25.96%   +0.1%     
==========================================
  Files          432      432             
  Lines        15869    15857     -12     
==========================================
+ Hits          4105     4118     +13     
+ Misses       11764    11739     -25
Impacted Files Coverage Δ
Pods/FirebaseCore/Firebase/Core/FIROptions.m 0% <ø> (ø) ⬆️
Rocket.Chat/API/Clients/SubscriptionsClient.swift 37.97% <0%> (ø) ⬆️
Pods/FirebaseCore/Firebase/Core/FIRApp.m 1.66% <0%> (+0.04%) ⬆️
...FirebaseCore/Firebase/Core/FIRComponentContainer.m 13.68% <0%> (+13.68%) ⬆️

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 d16a1b0...e1b4317. Read the comment docs.

Copy link
Member

@cardoso cardoso left a comment

Choose a reason for hiding this comment

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

👍

@rafaelks rafaelks merged commit 3c1c15a into develop Nov 8, 2018
@rafaelks rafaelks deleted the chore/pods_updates branch November 8, 2018 15:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants