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.

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

Test with Chainer v6 on CI #247

Merged
merged 2 commits into from
Nov 6, 2019

Conversation

disktnk
Copy link
Member

@disktnk disktnk commented Nov 6, 2019

Chainer v7rc1 includes ONNX-Chainer and this repository works with Chainer v6 or before. This PR removes jobs using Chainer preview version, and test only with Chainer v6.

@disktnk disktnk added this to the 1.5.1 milestone Nov 6, 2019
@codecov-io
Copy link

codecov-io commented Nov 6, 2019

Codecov Report

Merging #247 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #247   +/-   ##
=======================================
  Coverage   90.62%   90.62%           
=======================================
  Files          23       23           
  Lines        1621     1621           
=======================================
  Hits         1469     1469           
  Misses        152      152

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b8e8eb...38783ca. Read the comment docs.

@disktnk
Copy link
Member Author

disktnk commented Nov 6, 2019

/test

@pfn-ci-bot
Copy link
Collaborator

Successfully created a job for commit 38783ca:

Copy link
Member

@take-cheeze take-cheeze left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@take-cheeze take-cheeze merged commit 9ecbd31 into chainer:master Nov 6, 2019
@disktnk disktnk deleted the fix/ci-stop-chainer-pre branch November 6, 2019 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants