Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove unnecessary export on ONNX replace function test #8487

Merged
merged 1 commit into from Nov 29, 2019
Merged

Remove unnecessary export on ONNX replace function test #8487

merged 1 commit into from Nov 29, 2019

Conversation

disktnk
Copy link
Member

@disktnk disktnk commented Nov 29, 2019

t/o

@disktnk disktnk added the ONNX Related to ONNX label Nov 29, 2019
@disktnk
Copy link
Member Author

disktnk commented Nov 29, 2019

Jenkins, test this please!

@msakai
Copy link
Contributor

msakai commented Nov 29, 2019

Thanks! It looks good to me.
I think that I added this line in #8398 by mistake.

@chainer-ci
Copy link
Member

Jenkins CI test (for commit 9df38e8, target branch master) failed with status FAILURE.

@take-cheeze
Copy link
Member

Jenkins, test this please!

@chainer-ci
Copy link
Member

Jenkins CI test (for commit 9df38e8, target branch master) succeeded!

@take-cheeze take-cheeze added the st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes. label Nov 29, 2019
@mergify mergify bot merged commit c6614ad into chainer:master Nov 29, 2019
@disktnk disktnk deleted the fix/remove-test-files branch November 29, 2019 09:24
@kmaehashi kmaehashi added this to the v7.0.0 milestone Dec 5, 2019
@kmaehashi kmaehashi added the cat:test Test or CI related. label Dec 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shinh shinh shinh approved these changes

Assignees
No one assigned
Labels
cat:test Test or CI related. ONNX Related to ONNX st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes.
Projects
None yet
Milestone
v7.0.0
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@disktnk @msakai @chainer-ci @take-cheeze @shinh @kmaehashi