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

make sure not to call map on nil array #10219

Merged
merged 1 commit into from Aug 31, 2017
Merged

make sure not to call map on nil array #10219

merged 1 commit into from Aug 31, 2017

Conversation

ohayon
Copy link
Contributor

@ohayon ohayon commented Aug 31, 2017

Fixes #10214 where the test's failure's array is nil.

Copy link
Member

@KrauseFx KrauseFx left a comment

Choose a reason for hiding this comment

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

yess Array(...) ❤️

@ohayon ohayon merged commit 204d7fa into master Aug 31, 2017
@ohayon ohayon deleted the fix-snapshot-crash branch August 31, 2017 18:35
@fishcharlie
Copy link

@KrauseFx @ohayon Has this been deployed? Having the same issue.

@KrauseFx
Copy link
Member

Not yet

@fastlane-bot
Copy link

Congratulations! 🎉 This was released as part of fastlane 2.55.0 🚀

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.

None yet

5 participants