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

[WEEX-562][Android] task may be null ,should be check ,not try/catch #1425

Merged
merged 1 commit into from Aug 13, 2018

Conversation

Projects
None yet
3 participants
@lucky-chen
Contributor

lucky-chen commented Aug 13, 2018

task may be null,

  • should check before call toArray
  • try/catch is not a good idea
@weex-bot

This comment has been minimized.

weex-bot commented Aug 13, 2018

Fails
🚫 Failed to run assembleDebug task for android.
Messages
📖 android build verification finished.

Generated by 🚫 dangerJS

@YorkShen YorkShen merged commit 9eb436a into apache:master Aug 13, 2018

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment