-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
LGTM : )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution. Since the CI failed for unrelated error, can you retrigger the CI by pushing an empty commit
Using something like
git commit --allow-empty -m "Trigger notification"
windows CI is currently not working so we need to wait until it's resolved to trigger again. |
@mxnet-label-bot add [pr-awaiting-merge] |
@pldeepesh would you mind doing a rebase? git remote add upstream https://github.com/apache/incubator-mxnet
git pull upstream master --rebase
# resolve conflicts if any
git push --force |
@mxnet-label-bot update [pr-awaiting-testing] |
Merged. Thanks! @pldeepesh Would you like to submit a new PR to add your name in https://github.com/apache/incubator-mxnet/blob/master/CONTRIBUTORS.md ? |
Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments