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

Hotfix/v3.4.2 #77

Merged
merged 8 commits into from
Nov 12, 2019
Merged

Hotfix/v3.4.2 #77

merged 8 commits into from
Nov 12, 2019

Conversation

edugonzlz
Copy link
Contributor

No description provided.

@edugonzlz edugonzlz requested a review from radaloda November 12, 2019 07:57
self.downloadNext()
}
} else {
LogWarn("Al descargar la imagen,o se ha recibido un body vacio o no se se ha reconocido el tipo de imagen que es.")

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 135 characters (line_length)

if let currentRequest = ImageDownloader.queue[view], request.baseURL == currentRequest.baseURL {
self.setAnimated(image: imageGif, in: view)
}

Choose a reason for hiding this comment

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

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

}

self.setAnimated(image: finalImage, in: view)
}

Choose a reason for hiding this comment

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

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

self.downloadNext()
}
} else if let imageGif = try? response.gif() {

Choose a reason for hiding this comment

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

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)
Vertical Whitespace Violation: Limit vertical whitespace to a single empty line. Currently 2. (vertical_whitespace)

Copy link
Contributor

@radaloda radaloda left a comment

Choose a reason for hiding this comment

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

Good job! @edugonzlz

@radaloda radaloda merged commit 8cb198f into master Nov 12, 2019
@Alesete Alesete deleted the hotfix/v3.4.2 branch January 29, 2020 13:57
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