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

[iOS] Allow redundant arguments provided for callNative. #1987

Merged
merged 1 commit into from
Jan 2, 2019
Merged

[iOS] Allow redundant arguments provided for callNative. #1987

merged 1 commit into from
Jan 2, 2019

Conversation

wqyfavor
Copy link
Member

@wqyfavor wqyfavor commented Jan 2, 2019

First of all, thank you for your contribution!

Please makes sure that these checkboxes are checked before submitting your PR, thank you!

CheckList:

if isBugFix :

  • Make sure that you add at least one unit test for the bug which you had fixed.

elif isNewFeature :

  • Update API docs for the component.
  • Update/Add demo to demonstrate new feature.
  • Update test scripts for the feature.
  • Add unit tests for the feature.

@weex-bot
Copy link

weex-bot commented Jan 2, 2019 •
edited
Loading

Messages
📖 No android file has been changed.
📖 android build verification finished.

Generated by 🚫 dangerJS

doumafang
doumafang approved these changes Jan 2, 2019
View reviewed changes
Copy link
Member

@doumafang doumafang left a comment

Choose a reason for hiding this comment

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

ok

@doumafang doumafang merged commit ab262a1 into apache:master Jan 2, 2019
@wqyfavor wqyfavor deleted the allow-redundant-arguments branch January 3, 2019 05:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@doumafang doumafang doumafang approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@wqyfavor @weex-bot @doumafang