Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

avoid log npe #2856

Merged
merged 1 commit into from Aug 28, 2019
Merged

avoid log npe #2856

merged 1 commit into from Aug 28, 2019

Conversation

Darin726
Copy link
Contributor

No description provided.

@Darin726 Darin726 force-pushed the feature/fix-npe-for-log-printl branch from d484378 to 5c212db Compare August 28, 2019 08:07
lucky-chen
lucky-chen approved these changes Aug 28, 2019
View reviewed changes
Copy link
Contributor

@lucky-chen lucky-chen left a comment

Choose a reason for hiding this comment

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

ok

@weex-bot
Copy link

Warnings
⚠️ No Changelog changes! - Can you add a Changelog? To do so,append your changes to the changelog.md
⚠️ Current pr not bind the milestone
⚠️ If your PR is about fixing a bug excluding crash the code,you should add the demo link in the PR description. Demo link: http://dotwe.org/vue
Messages
📖 danger test finished.

Generated by 🚫 dangerJS against 5c212db

@lucky-chen lucky-chen merged commit 8722cf3 into apache:master Aug 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lucky-chen lucky-chen lucky-chen approved these changes

@YorkShen YorkShen Awaiting requested review from YorkShen YorkShen is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Darin726 @weex-bot @lucky-chen