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

Fix compile error in test target when running on 32 bit device #978

Merged
merged 1 commit into from
Jun 29, 2018

Conversation

danj-stripe
Copy link
Contributor

Summary

Added a cast to CGFloat

Motivation

Warning about implicit conversion that our compiler settings elevate to an error.

Testing

Compiled/ran test suite on both iPhone 5 & iPhone 6

Warning about implicit conversion that our compiler settings elevate to an error.
@danj-stripe
Copy link
Contributor Author

This test was recently added by me, in #954

@joeydong-stripe
Copy link
Contributor

While we're at it, I notice now that those color tests also don't compile on iOS 9.

@danj-stripe
Copy link
Contributor Author

@joeydong-stripe I noticed that too. I figured it could be a separate PR, since it's a separate issue.

@danj-stripe danj-stripe merged commit aa2827d into master Jun 29, 2018
@danj-stripe danj-stripe deleted the danj/bugfix/compile-error-iphone5 branch June 29, 2018 23:28
csabol-stripe added a commit that referenced this pull request Apr 26, 2022
* Adds bank name to icon code mapping

* Set icon based on name

* Adds new default icon and corner radius

* Revert unneeded test changes

* Update tests for default

* Fix corner radius
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants