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

Merge Release/0.28 into master #2968

Merged
merged 24 commits into from Oct 24, 2019
Merged

Merge Release/0.28 into master #2968

merged 24 commits into from Oct 24, 2019

Conversation

YorkShen
Copy link
Contributor

Brief Description of the PR

Checklist

  • Demo:
  • Documentation:

@weex-bot
Copy link

weex-bot commented Oct 15, 2019 •
edited

Warnings
⚠️ 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
⚠️ If you update the code, maybe you should update the documentation and add the documentation link in the PR description. here is the guide about how to contribute documentation: https://github.com/apache/incubator-weex/blob/master/CONTRIBUTING.md#contribute-code-or-document

Generated by 🚫 dangerJS against cce4063

@YorkShen YorkShen marked this pull request as ready for review October 18, 2019 07:36
lucky-chen
lucky-chen approved these changes Oct 24, 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.

lgtm

@lucky-chen lucky-chen merged commit 8922b1e into master Oct 24, 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

@Darin726 Darin726 Awaiting requested review from Darin726

@gubaojian gubaojian Awaiting requested review from gubaojian

@wqyfavor wqyfavor Awaiting requested review from wqyfavor wqyfavor 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
@YorkShen @weex-bot @lucky-chen